Re: tellico 3.0.2-1

[Brian Inglis]

On 2017-07-18 19:14, Yaakov Selkowitz wrote:
> The following packages have been uploaded to the Cygwin distribution:
> * tellico-3.0.2-1
> Tellico makes it easy to track your books, videos, music, even your wine 
> and anything else. A simple and intuitive interface shows cover images, 
> groupings, and any detail you want. Grab information from many popular 
> Internet sites, including IMDB.com, Amazon.com, and most libraries.
> This is an update to the latest upstream release, which is now KF5-based.

Announcement appeared on cygwin-announce but not on cygwin list.

-- 
Take care. Thanks, Brian Inglis, Calgary, Alberta, Canada

Re: [Attention Maintainer] man-db

[Ken Brown]

On 7/19/2017 1:53 PM, Achim Gratz wrote:

Achim Gratz writes:

This package currently has a perpetual postinstall script to keep the
database indexed, but doesn't provide for a way to create the initial
database during install.


This is done deliberately, because the creation of the initial database 
can be very time consuming.  See the release announcement:


  https://sourceware.org/ml/cygwin-announce/2016-10/msg00037.html

FWIW, I agree with Yaakov's decision here.  I think many (most?) users 
would be very annoyed if they had to wait for mandb to finish its 
initial run.  And I suspect the Cygwin list would be flooded with 
complaints that setup is hanging.


Ken

Re: [GOLDSTAR] Re: fflush(NULL) clobbers input streams

[Andrew Schulman]

> > Achim, thanks for the testcase.  Ken, thanks for bisecting.  You're
> > my hero!
> > 
> > I reverted the patch and created new developer snapshots.
> 
> I forgot the gold star.  Thanks again, Ken!  

Awarded!

https://cygwin.com/goldstars/#AG
https://cygwin.com/goldstars/#KB

Re: [GOLDSTAR] Re: fflush(NULL) clobbers input streams

[Ken Brown]

On 7/19/2017 1:03 PM, Corinna Vinschen wrote:

On Jul 19 10:45, Corinna Vinschen wrote:

On Jul 18 22:16, Ken Brown wrote:

On 7/18/2017 6:41 PM, Ken Brown wrote:

On 7/18/2017 3:21 PM, Achim Gratz wrote:

Corinna Vinschen writes:

Oh, and a plain C testcase, of course...


Call that fflush.c:
[...]
Then run:

gcc fflush.c -o fflush
cat fflush.c | ./fflush 2>/dev/null > fflush.out
diff fflush.c fflush.out

If things are working, fflush.c and fflush.out should be identical.
Currently only the first line makes it into fflush.out on Cygwin.


I've checked that the 2017-03-08 snapshot is good and the 2017-03-10
snapshot is bad.  I'll bisect further when I get a chance


Done:

44b1746a41921533d27aca414a9188314cb725b6 is the first bad commit
commit 44b1746a41921533d27aca414a9188314cb725b6
Author: Corinna Vinschen 
Date:   Fri Mar 10 20:21:09 2017 +0100

 errno: Stop using _impure_ptr->_errno completely

 We use errno AKA _REENT->_errno since the last century and only set
 _impure_ptr->_errno for backward compat.  Stop that.  Also, remove
 the last check for _impure_ptr->_errno in Cygwin code.

Ken


Achim, thanks for the testcase.  Ken, thanks for bisecting.  You're
my hero!

I reverted the patch and created new developer snapshots.


I forgot the gold star.  Thanks again, Ken!


Glad I could do it.  And thanks for the hero comment.  You made my day.

Ken

[Attention Maintainer] man-db

[Achim Gratz]

This package currently has a perpetual postinstall script to keep the
database indexed, but doesn't provide for a way to create the initial
database during install.  I suggest to create another sub-package
(perhaps called man-db-index) that simply creates an empty file
/var/cache/man-db/index.dat on installation, so that the perpetual
install script will trigger.  I would prefer if that file was created by
a normal postinstall script instead of directly provided in the package
since otherwise deinstallation of man-db-index would delete the
database.


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

SD adaptations for KORG EX-800 and Poly-800MkII V0.9:
http://Synth.Stromeko.net/Downloads.html#KorgSDada

Re: fflush(NULL) clobbers input streams

[Achim Gratz]

Corinna Vinschen writes:
> Achim, thanks for the testcase.

I really only pulled it out of the configure script.
:-)


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

Perpetual postinstall scripts

[Achim Gratz]

We're starting to get more and more of these, which is fine, since that
was why they were introduced.  However, I'd like to remind everybody to
not use bash for any postinstall scripts unless you absolutely have to
and especially not for perpetual postinstall scripts.  It is only
slightly less troublesome to script for dash if your script doesn't use
any bashisms, either set PATH=/usr/bin at the beginning of the script or
use absolute path names for all executables.

Since a lot of the perpetual postinstall scripts check a condition to
decide if they should do anything, I'd also repeat my recommendation to
avoid forking subprocesses as much as possible (use builtins instead of
/usr/bin/test for instance) and hand off to terminal executables via
exec.  This drastically increases the chances of things still working OK
if something went wrong with rebasing.


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

Re: fflush(NULL) clobbers input streams

[Achim Gratz]

Ken Brown writes:
>> I've checked that the 2017-03-08 snapshot is good and the 2017-03-10
>> snapshot is bad.  I'll bisect further when I get a chance
>
> Done:
>
> 44b1746a41921533d27aca414a9188314cb725b6 is the first bad commit
> commit 44b1746a41921533d27aca414a9188314cb725b6
> Author: Corinna Vinschen 
> Date:   Fri Mar 10 20:21:09 2017 +0100
>
> errno: Stop using _impure_ptr->_errno completely
>
> We use errno AKA _REENT->_errno since the last century and only set
> _impure_ptr->_errno for backward compat.  Stop that.  Also, remove
> the last check for _impure_ptr->_errno in Cygwin code.

Thank you!  Well, that would have been the last commit I'd suspected.

Andrew, are you perhaps having a goldstar in the vault for Mr. Brown?


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

Re: fflush(NULL) clobbers input streams

[Corinna Vinschen]

On Jul 18 22:16, Ken Brown wrote:
> On 7/18/2017 6:41 PM, Ken Brown wrote:
> > On 7/18/2017 3:21 PM, Achim Gratz wrote:
> > > Corinna Vinschen writes:
> > > > Oh, and a plain C testcase, of course...
> > > 
> > > Call that fflush.c:
> > > [...]
> > > Then run:
> > > 
> > > gcc fflush.c -o fflush
> > > cat fflush.c | ./fflush 2>/dev/null > fflush.out
> > > diff fflush.c fflush.out
> > > 
> > > If things are working, fflush.c and fflush.out should be identical.
> > > Currently only the first line makes it into fflush.out on Cygwin.
> > 
> > I've checked that the 2017-03-08 snapshot is good and the 2017-03-10
> > snapshot is bad.  I'll bisect further when I get a chance
> 
> Done:
> 
> 44b1746a41921533d27aca414a9188314cb725b6 is the first bad commit
> commit 44b1746a41921533d27aca414a9188314cb725b6
> Author: Corinna Vinschen 
> Date:   Fri Mar 10 20:21:09 2017 +0100
> 
> errno: Stop using _impure_ptr->_errno completely
> 
> We use errno AKA _REENT->_errno since the last century and only set
> _impure_ptr->_errno for backward compat.  Stop that.  Also, remove
> the last check for _impure_ptr->_errno in Cygwin code.
> 
> Ken

Achim, thanks for the testcase.  Ken, thanks for bisecting.  You're
my hero!

I reverted the patch and created new developer snapshots.


Thanks,
Corinna

-- 
Corinna Vinschen  Please, send mails regarding Cygwin to
Cygwin Maintainer cygwin AT cygwin DOT com
Red Hat


signature.asc
Description: PGP signature