[BUG] lftp 4.3.6

[BUG] lftp 4.3.6

[Achim Gratz]

There is an insidious bug that affects at least lftp on the latest
cygwin snapshot as well as the 1.7.15 release.  Prepare and run an lftp
script like this:

perl -e 'foreach (1..100){' \
     -e 'print "ls -r file:/", ' \
     -e '(map {"$_"x10. "/"}(a..z)), "\n"}' \
     > test.lftp
lftp -f test.lftp |& grep -v ls:

Observe that lftp splits the commands at a 4kiB boundary (a read
buffer?) and gives an error on the part over in the next buffer as if
the buffer boundary had been an EOL (which also means that it tried to
execute the incomplete command).

Now, the script I've shown above is harmless and in fact doesn't do
anything.  I haven't been so lucky, in my case the script got cut
exactly after an

rm -r /cygdrive/c

In the past I've cursed the disk in my computer at work for being so
slow, but today I was glad that it wasn't zippy.  Still, the rest of the
day I've had to clean up the damage.

Under Linux i can't get lftp 4.3.3 to misbehave in the same manner even
with much larger command lines and files, so it is either something
introduced with a newer version of lftp or unique to lftp on Cygwin.


Regards,
Achim.
-- 
+<[Q+ Matrix-12 WAVE#46+305 Neuron microQkb Andromeda XTk Blofeld]>+

Wavetables for the Terratec KOMPLEXER:
http://Synth.Stromeko.net/Downloads.html#KomplexerWaves


--
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

Re: [BUG] lftp 4.3.6

[Achim Gratz]

Achim Gratz writes:
> perl -e 'foreach (1..100){' \
>      -e 'print "ls -r file:/", ' \
>      -e '(map {"$_"x10. "/"}(a..z)), "\n"}' \
>      > test.lftp

Sorry, there's a space missing:

perl -e 'foreach (1..100){' \
     -e 'print "ls -r file:/", ' \
     -e '(map {"$_"x10 . "/"}(a..z)), "\n"}' \
     > test.lftp


Regards,
Achim.
-- 
+<[Q+ Matrix-12 WAVE#46+305 Neuron microQkb Andromeda XTk Blofeld]>+

Wavetables for the Waldorf Blofeld:
http://Synth.Stromeko.net/Downloads.html#BlofeldUserWavetables


--
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

Re: [BUG] lftp 4.3.6

[Achim Gratz]

Achim Gratz <Stromeko <at> nexgo.de> writes:
> Under Linux i can't get lftp 4.3.3 to misbehave in the same manner even
> with much larger command lines and files, so it is either something
> introduced with a newer version of lftp or unique to lftp on Cygwin.

The previous lftp version 4.3.3 on Cygwin also does not show the bug.  I've
reported the bug to the lftp mailing list as well.  I'm investigating further
with a local compile...


Regards,
Achim.




--
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

Re: [BUG] lftp 4.3.6

[Achim Gratz]

Achim Gratz <Stromeko <at> NexGo.DE> writes:
> The previous lftp version 4.3.3 on Cygwin also does not show the bug.  I've
> reported the bug to the lftp mailing list as well.  I'm investigating further
> with a local compile...

I've compiled both the 4.3.3 and 4.3.6 version using the same settings, so they
have the same libraries etc. -- 4.3.3 is still good and 4.3.6 has the bug. 
Let's see what upstream has to say about it, meanwhile I'd think that 4.3.6
should be pulled from distribution...


Regards,
Achim.


--
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

Re: [BUG] lftp 4.3.6

[Achim Gratz]

Achim Gratz <Stromeko <at> NexGo.DE> writes:
> I've compiled both the 4.3.3 and 4.3.6 version using the same settings, so they
> have the same libraries etc. -- 4.3.3 is still good and 4.3.6 has the bug. 
> Let's see what upstream has to say about it, meanwhile I'd think that 4.3.6
> should be pulled from distribution...

Upstream has released 4.3.7 which fixes the bug.  Yay!


Regards,
Achim.




--
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

Re: [BUG] lftp 4.3.6

[Andrew Schulman]

> Achim Gratz <Stromeko <at> NexGo.DE> writes:
> > I've compiled both the 4.3.3 and 4.3.6 version using the same settings, so they
> > have the same libraries etc. -- 4.3.3 is still good and 4.3.6 has the bug. 
> > Let's see what upstream has to say about it, meanwhile I'd think that 4.3.6
> > should be pulled from distribution...
> 
> Upstream has released 4.3.7 which fixes the bug.  Yay!

Thanks for chasing that down.  I'll package and release 4.3.7 shortly.  Andrew.


--
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

Re: [BUG] lftp 4.3.6

[Achim Gratz]

Andrew Schulman writes:
>> Upstream has released 4.3.7 which fixes the bug.  Yay!
>
> Thanks for chasing that down.  I'll package and release 4.3.7 shortly.  Andrew.

Thank you.  When you're at it, please remove the README~ from the patch
file... :-)


Regards,
Achim.
-- 
+<[Q+ Matrix-12 WAVE#46+305 Neuron microQkb Andromeda XTk Blofeld]>+

SD adaptations for Waldorf Q V3.00R3 and Q+ V3.54R2:
http://Synth.Stromeko.net/Downloads.html#WaldorfSDada


--
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

Re: [BUG] lftp 4.3.6

[Andrew Schulman]

> Andrew Schulman writes:
> >> Upstream has released 4.3.7 which fixes the bug.  Yay!
> >
> > Thanks for chasing that down.  I'll package and release 4.3.7 shortly.  Andrew.
> 
> Thank you.  When you're at it, please remove the README~ from the patch
> file... :-)

OK, sorry.  Most annoying emacs feature that yet I've never bothered to turn
off.  I always try to cut those out, but sometimes they slip through.


--
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

Re: [BUG] lftp 4.3.6

[Achim Gratz]

Andrew Schulman writes:
> OK, sorry.  Most annoying emacs feature that yet I've never bothered to turn
> off.  I always try to cut those out, but sometimes they slip through.

No need to be sorry, but maybe we could convince Yaakov not to package
autosave files or at least give a warning about their presence?


Regards,
Achim.
-- 
+<[Q+ Matrix-12 WAVE#46+305 Neuron microQkb Andromeda XTk Blofeld]>+

SD adaptations for Waldorf Q V3.00R3 and Q+ V3.54R2:
http://Synth.Stromeko.net/Downloads.html#WaldorfSDada


--
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

Re: [BUG] lftp 4.3.6

[Yaakov (Cygwin/X)]

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

On 2012-05-30 14:32, Achim Gratz wrote:
> Andrew Schulman writes:
>> OK, sorry.  Most annoying emacs feature that yet I've never bothered to turn
>> off.  I always try to cut those out, but sometimes they slip through.
>
> No need to be sorry, but maybe we could convince Yaakov not to package
> autosave files or at least give a warning about their presence?

As in the attached?


Yaakov


[-- Attachment #2: 0001-pkg-exclude-text-editor-backup-files-in-CYGWIN-PATCH.patch --]
[-- Type: application/x-itunes-itlp, Size: 1033 bytes --]

[-- 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

Re: [BUG] lftp 4.3.6

[Achim Gratz]

Yaakov (Cygwin/X <yselkowitz <at> users.sourceforge.net> writes:
> As in the attached?

I'd think you should quote the exclude regex like '*~' or \*~, but otherwise
this certainly looks appealing.

Sorry for bringing up another issue: is it intended that KEEPDIRS produces
.keep-<package> files in the install?  I thought that these should not end up in
the installer, but only prevent the removal of empty directories, but maybe I
misunderstood.


Regards,
Achim.


--
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

Re: [BUG] lftp 4.3.6

[Achim Gratz]

Achim Gratz <Stromeko <at> NexGo.DE> writes:
> I'd think you should quote the exclude regex like '*~' or \*~, but otherwise
> this certainly looks appealing.

I've gone with the \*~ variant and it works like a charm.


Regards,
Achim.


--
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

Re: [BUG] lftp 4.3.6

[Achim Gratz]

Andrew Schulman writes:
> Thanks for chasing that down.  I'll package and release 4.3.7 shortly.  Andrew.

I just see your RFU over in cygwin apps... Please leave 4.3.3 as
previous and remove 4.3.6, thanks.


Regards,
Achim.
-- 
+<[Q+ Matrix-12 WAVE#46+305 Neuron microQkb Andromeda XTk Blofeld]>+

Factory and User Sound Singles for Waldorf rackAttack:
http://Synth.Stromeko.net/Downloads.html#WaldorfSounds


--
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

Re: [BUG] lftp 4.3.6

[Andrew Schulman]

> Andrew Schulman writes:
> > Thanks for chasing that down.  I'll package and release 4.3.7 shortly.  Andrew.
> 
> I just see your RFU over in cygwin apps... Please leave 4.3.3 as
> previous and remove 4.3.6, thanks.

Done.


--
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