From: Jon Turney <jon.turney@dronecode.org.uk>
To: cygwin-apps@cygwin.com
Subject: Re: setup.hint documentation issues
Date: Wed, 17 Feb 2016 14:31:00 -0000 [thread overview]
Message-ID: <56C48426.50904@dronecode.org.uk> (raw)
In-Reply-To: <877fid3cle.fsf@Rainer.invalid>
On 09/02/2016 19:31, Achim Gratz wrote:
> Jon Turney writes:
>> I'd suggest that double-quoting of those keys is made mandatory, and
>> embedded double-quotes are forbidden, as this permits simpler
>> processing of this text, lexing character by character.
>
> OK, although we might need some sort of escaping in the long run.
Yes, I have no problem with later adding escaping e.g. using '\', if
needed, since that can written with a character by character lexer.
But writing a replacement setup.hint parser, while maintaining
bug-for-bug compatibility with the existing behaviour, requires a
line-by-line processing with various transformations...
>> upset knows enough to omit packages which have no install tarfiles
>> (i.e. are source-only) from from setup.ini, irrespective of 'skip'.
>>
>> However, the presence of 'skip' also causes the package to be omitted
>> from the HTML package list.
>
> I could be wrong, but it seems it may have been intended to deal with
> packages that became obsolete, but the previous version was still kept.
Any package marked 'skip' is completely omitted from setup.ini, so setup
won't do anything with it, so I don't think that can be the use for it.
>> I think cygport's behaviour has changed over time, but currently will
>> mark source-packages as 'skip', however there are several packages
>> that are source-only (e.g. attica), that are missing 'skip'.
>
> So the proposal is to remove skip or make it mandatory for source-only
> packages?
I'm not sure. I think I tend towards removing it, since it doesn't add
any information.
But currently cygport generates a setup.hint for source-only package
which only contains 'skip'. I'm not sure that's the best idea, as
having no sdesc, etc. means we can't describe the source package.
next prev parent reply other threads:[~2016-02-17 14:31 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-02-09 13:18 Jon Turney
2016-02-09 17:10 ` Corinna Vinschen
2016-02-17 14:23 ` Jon Turney
2016-02-17 17:34 ` Corinna Vinschen
2016-03-29 18:36 ` Jon Turney
2016-02-09 19:31 ` Achim Gratz
2016-02-17 14:31 ` Jon Turney [this message]
2016-02-17 18:03 ` Achim Gratz
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=56C48426.50904@dronecode.org.uk \
--to=jon.turney@dronecode.org.uk \
--cc=cygwin-apps@cygwin.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).