[PATCH] 'rebase' uninitialised variable (was: Cygwin64: Rebase problems)

[David Stacey <drstacey@tiscali.co.uk>]

[-- Attachment #1: Type: text/plain, Size: 794 bytes --]

On 02/11/2013 11:06, David Stacey wrote:
> I'm trying to reinstall 64-bit Cygwin, using a snapshot taken on 
> 2013-10-24. The post-install step goes into an infinite loop whilst 
> running 'texlive-collection-basic.sh': perl is attempting to load some 
> modules (IO.dll, MD5.dll, Fcnt.dll, Cwd.dll), which fail with rebase 
> errors. The script waits five seconds before trying (and failing) 
> again; this repeats ad infinitum.

I am pleased to report that I have tracked down the cause of my rebase 
woes to an uninitialised variable in the 'rebase' package. A simple 
patch (attached) fixes the problem.

Jason Tishler: As rebase maintainer, if you agree with my diagnosis then 
please could you make new versions of 'rebase' containing the fix for 
both architectures. Thank you.

Dave.


[-- Attachment #2: rebase-4.4.0.1-1.src.patch --]
[-- Type: text/plain, Size: 593 bytes --]

--- origsrc/rebase/rebase.c	2013-03-19 17:33:54.000000000 +0000
+++ src/rebase/rebase.c	2013-11-29 18:32:08.760089000 +0000
@@ -861,6 +861,7 @@ collect_image_info (const char *pathname
   img_info_list[img_info_size].slot_size
     = roundup2 (img_info_list[img_info_size].size, ALLOCATION_SLOT);
   img_info_list[img_info_size].flag.needs_rebasing = 1;
+  img_info_list[img_info_size].flag.cannot_rebase = 0;
   /* This back and forth from POSIX to Win32 is a way to get a full path
      more thoroughly.  For instance, the difference between /bin and
      /usr/bin will be eliminated. */


[-- Attachment #3: Type: text/plain, Size: 218 bytes --]

--
Problem reports:       http://cygwin.com/problems.html
FAQ:                   http://cygwin.com/faq/
Documentation:         http://cygwin.com/docs.html
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple