This is the mail archive of the
cygwin-apps
mailing list for the Cygwin project.
Re: [RFC] incremental rebase
- From: Ken Brown <kbrown at cornell dot edu>
- To: cygwin-apps at cygwin dot com
- Date: Tue, 02 Dec 2014 17:20:52 -0500
- Subject: Re: [RFC] incremental rebase
- Authentication-results: sourceware.org; auth=none
- References: <87k32vjm3i dot fsf at Rainer dot invalid> <87y4r1r7rd dot fsf at Gertrud dot fritz dot box> <87d287j8kh dot fsf at Gertrud dot fritz dot box> <20141202205040 dot GA11162 at calimero dot vinschen dot de> <20141202205718 dot GB11162 at calimero dot vinschen dot de>
On 12/2/2014 3:57 PM, Corinna Vinschen wrote:
On Dec 2 21:50, Corinna Vinschen wrote:
On Nov 28 22:53, Achim Gratz wrote:
New versions of the setup programs and incremental autorebase package
are uploaded, plus my original texlive postinstall (not modified for the
new setup) for Ken to look at.
--8<---------------cut here---------------start------------->8---
wget="wget -rxnH http://cygwin.stromeko.net/
$wget/x86/setup-x86.exe
$wget/x86_64/setup-x86_64.exe
$wget/noarch/release/_incautorebase/_incautorebase-00009-1.tar.xz
$wget/noarch/release/_incautorebase/_incautorebase-00007-1-src.tar.xz
$wget/noarch/release/_incautorebase/setup.hint
$wget/noarch/release/zzz_texlive-postinstall/zzz_texlive-postinstall-00001-2.tar.bz2
$wget/noarch/release/zzz_texlive-postinstall/zzz_texlive-postinstall-00001-2-src.tar.bz2
$wget/noarch/release/zzz_texlive-postinstall/setup.hint
--8<---------------cut here---------------end--------------->8---
This version implements some fixes/improvements discussed on this list
plus the implementation of dash as an extra shell type. Not implemented
in the pre-compiled executables is the additional suffix ".cmd" for
batch files, but this is already in the source code at:
http://repo.or.cz/w/cygwin-setup.git
Comments are welcome.
This is neat. What I'm wondering about is if we still need to run the
peflags stuff. The gcc-created executables are TS-aware for a long time
now.
Uh, sorry, I was only taking about _incautorebase if that wasn't clear.
I was just looking into the scripts and I think I like what I see.
It's just the peflags thingy I'm not so sure about. At least on 64 bit
it's totally unnecessary. Can this be made conditional, perhaps, calling
peflags_do only on 32 bit?
There's one small problem with _incautorebase as it stands: When the
package is first installed, the script 001_incautorebase.dash will get
run *after* all 0p_* scripts have been run. But it needs to run first.
Something like the following should fix this:
--- 0p_incautorebase.dash.orig 2014-11-28 02:31:49.000000000 -0500
+++ 0p_incautorebase.dash 2014-12-02 17:06:23.834840800 -0500
@@ -1,7 +1,11 @@
#!/bin/dash
export PATH=/bin
-if [ -x /etc/postinstall/001_incautorebase.dash ] ; then
- echo Incremental autorebase has just been installed, postponing.
+initscript=/etc/postinstall/001_incautorebase.dash
+if [ -x ${initscript} ] ; then
+ echo "Initializing incremental autorebase."
+ if ${initscript} ; then
+ mv -f ${initscript} ${initscript}.done
+ fi
else
rebaselst update rebase peflags
fi
Ken