pkgsrc-WIP-discuss archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: pkglint -Wall



Iain Hibbert wrote:
I wonder if the message that 'pkglint -Wall' gives might be more clearly
understood if either it just said 'use :Q you fool' as it does normally or
if it used the escape word somewhere?

Hmm, this reminds me of a particular gcc warning:

    warning: assignment discards qualifiers from pointer target type

I had sent them a bug report because there was no hint on how to fix this warning properly. Not even the word ``const'' appeared in it. Now exactly the same happened to me. :)

I have changed the wording of the message to the following:

    WARN: builtin.mk:225--243: Please use ${HEAD:Q} instead of ${HEAD}
          and make sure the variable appears outside of any quoting
          characters.

If the variable already appears outside of any quoting characters, the last two lines from above are of course omitted. The warning text is pretty long, but I couldn't think of a shorter one that would be similarly exact. If you know one, please tell me.

Roland


-------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc. Do you grep through log files
for problems?  Stop!  Download the new AJAX search engine that makes
searching your log files as easy as surfing the  web.  DOWNLOAD SPLUNK!
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=103432&bid=230486&dat=121642
_______________________________________________
pkgsrc-wip-discuss mailing list
pkgsrc-wip-discuss%lists.sourceforge.net@localhost
https://lists.sourceforge.net/lists/listinfo/pkgsrc-wip-discuss



Home | Main Index | Thread Index | Old Index