tech-pkg archive
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
Re: Combining NOT_FOR_* and BROKEN_ON_*
Thomas Klausner <wiz%NetBSD.org@localhost> writes:
1> Hi!
>
> We currently have two settings to disable packages for particular
> platforms:
>
> NOT_FOR_* for packages where it "doesn't make sense" for them to be
> built on a platform, and BROKEN_ON_* for packages where it does, but
> they don't build for some reason.
>
> I understand the difference between the two, but I don't see it as a
> useful difference to make - the result is the same, we don't try
> building on that platform.
It documents for the reader whether there is something to be done
(BROKEN_ON), or whether there is not (NOT_FOR).
> It also has lead to a couple discussions already if one or the other
> is more appropriate for a particular package, none of which I remember
> as being useful.
Those would be avoided by people using BROKEN_ON in the first place :-)
And, if simply fixed, would be like a vast number of "this isn't quite
right" comments. (I make mistakes all the time, and sometimes people
just fix them, and sometimes people tell me, and I try to fix them
quickly.)
There seems to be a culture of "upstream is deliberately indifferent to
this platform (or to any computer older than 5 years old that lacks 16
GB of RAM, or whatever)" beyond pkgsrc, and within pkgsrc "mark it
NOT_FOR, accepting upstream's indifference as valid".
> To avoid the not very useful discussions, I suggest removing either of
> them; probably BROKEN_ON_*, but I don't really care.
I remember an earlier comment that BROKEN_ON was bad, because it hid the
failure and lacked any detection that a package might have been fixed.
This objection did not apply to NOT_FOR, more or less. However, we
don't seem to have anybody doing bulk builds ignoring BROKEN_ON, so that
may be a point without a purpose.
All in all, I don't think it's a good idea to merge these, but I also
can't say that it will matter much in the end.
Home |
Main Index |
Thread Index |
Old Index