public inbox for cygwin-apps@cygwin.com
 help / color / mirror / Atom feed
* [BUG] mksetupini
@ 2017-10-25 17:37 Achim Gratz
  2017-10-25 19:43 ` Jon Turney
  0 siblings, 1 reply; 2+ messages in thread
From: Achim Gratz @ 2017-10-25 17:37 UTC (permalink / raw)
  To: cygwin-apps


Consider the following situation: an old package with setup.hint that
has dependency to "abc" and a new package version using versioned hint
files without said dependency.  Not the old package files get deleted,
but setup.hint stays in the directory.  Running mksetupini correctly
determines that the setup.hint file needs to be removed, but still
generates a setup.ini that makes the package depend on "abc".

The expected behaviour of course would be that it doesn't use the
information in that file (which no longer exists at the end of the run
anyway).


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

Factory and User Sound Singles for Waldorf Q+, Q and microQ:
http://Synth.Stromeko.net/Downloads.html#WaldorfSounds

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [BUG] mksetupini
  2017-10-25 17:37 [BUG] mksetupini Achim Gratz
@ 2017-10-25 19:43 ` Jon Turney
  0 siblings, 0 replies; 2+ messages in thread
From: Jon Turney @ 2017-10-25 19:43 UTC (permalink / raw)
  To: cygwin-apps

On 25/10/2017 18:37, Achim Gratz wrote:
> Consider the following situation: an old package with setup.hint that
> has dependency to "abc" and a new package version using versioned hint
> files without said dependency.  Not the old package files get deleted,
> but setup.hint stays in the directory.  Running mksetupini correctly
> determines that the setup.hint file needs to be removed, but still
> generates a setup.ini that makes the package depend on "abc".
> 
> The expected behaviour of course would be that it doesn't use the
> information in that file (which no longer exists at the end of the run
> anyway).
   Yeah, this edge case isn't handled correctly:  There's a note in the 
source code [1] that basically says "XXX: do complex stuff here".

I believe If you run mksetupini again, you'll get the correct result, as 
the unneeded setup.hint has been removed.

Since I want to retire setup.hin support fairly soon, I'm not really 
planning on fixing this...

[1] 
https://cygwin.com/git/gitweb.cgi?p=cygwin-apps/calm.git;a=blob;f=calm/package.py;h=ed4cf009ce98c00f9182818353cc454f0c9525b9;hb=HEAD#l875

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2017-10-25 19:43 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-10-25 17:37 [BUG] mksetupini Achim Gratz
2017-10-25 19:43 ` Jon Turney

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).