Sean Whitton writes ("Bug#857316: dgit: please make --build-products-dir
configurable"):
> On Thu 09 Mar 2017 at 11:58PM +0100, Mattia Rizzolo wrote:
> > I build my packages with pbuilder, and it is set up to put the results
> > in ~/pbuilder/results/$distribution/$architecture/. I'd like dgit to go
> > look for the .changes to upload there, and I currently accomplish that
> > by passing `--build-products-dir ~/pbuilder/result/sid/amd64` to
> > `dgit push`. I'd like to configure that statically, so I don't have to
> > keep specifying it.
>
> Here are patches implementing this.
Thanks.
Why does the setting of $buildproductsdir need to be done in
build_or_push_prep_early, before pushing or notpushing ? I guess it
wouldn't make sense to have different settings, but the business with
localising $access_forpush is a wrinkle which is best avoided where
possible.
So I think it might be better to introduce build_or_push_prep,
which I guess would be called in prep_push after pushing and in
build_prep after build_prep_early.
Style: I normally spell:
+ my $bpd_from_cfg = access_cfg('build-products-dir', 'RETURN-UNDEF');
+ $buildproductsdir = $bpd_from_cfg // '..';
like this:
+ $buildproductsdir = access_cfg('build-products-dir', 'RETURN-UNDEF')
// '..';
And then,
+ if (!defined $buildproductsdir) {
* $buildproductsdir = access_cfg('build-products-dir', 'RETURN-UNDEF')
// '..';
+ }
could be spelled
* $buildproductsdir //= access_cfg('build-products-dir', 'RETURN-UNDEF') //
'..';
or
* $buildproductsdir //= access_cfg('build-products-dir', 'RETURN-UNDEF');
* $buildproductsdir //= '..';
I don't much like the new test case because it's a whole new test case
just for this. I think it would be better to add the configuration
option for a cross-section of existing tests.
Probably, that would be,
* introduce a new t-buildproductsdir-config subroutine
* add it to the top of a selection of existing tests
Do you want me to do that ?
Thanks,
Ian.
--
Ian Jackson <[email protected]> These opinions are my own.
If I emailed you from an address @fyvzl.net or @evade.org.uk, that is
a private address which bypasses my fierce spamfilter.
