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

Re: Buildd & binary-indep

Russ Allbery <rra@debian.org> writes:

> Raphael Hertzog <hertzog@debian.org> writes:
>> Well, we specified DEB_BUILD_OPTIONS space separated because build flags
>> frequently embed commas. Shall we not take the same decision
>> preemptively here?
> Most fields that take multiple values use commas.  We do have another that
> takes spaces (Architecture), but I would much rather use commas.
> If the field is space-separated, it can't be folded.  If you want it to be
> folded, it has to be comma-separated (unless you're proposing introducing
> yet a *fourth* syntax for control fields, which seems like a bad idea).
>> Also if we want to have lists as values for some "features" â?? like
>> "disable-hardening=pie,relro" â?? we'd be better with using space
>> separator.
> I think that would be a bad idea.  If we need to provide data that takes
> additional options, we should add another field, since we already have a
> syntax for that.  Not add key-value pairs inside the value of something
> that's already a key-value pair.

Do we need this in debian/control?

Why not debian/source/options or debian/source/hardening if it is a more
special feature that requires extra args?

I wouldn't even mind putting the "build-arch" flag in there too. Dpkg
already has options in there that affect how packages are build (e.g. I
use it to ignore the ccache dir when building the source package so it
can be kept across clean calls). That wouldn't require any policy
change, maybe that would make progress possible.


Reply to: