[PATCH stable] debian: use a wildcard target for docker-debian-$codename

Sean Farley sean at farley.io
Mon Apr 24 17:12:38 EDT 2017


Anton Shestakov <av6 at dwimlabs.net> writes:

> On Sat, 22 Apr 2017 14:19:32 -0400
> Augie Fackler <raf at durin42.com> wrote:
>
>> # HG changeset patch
>> # User Augie Fackler <augie at google.com>
>> # Date 1492879857 14400
>> #      Sat Apr 22 12:50:57 2017 -0400
>> # Branch stable
>> # Node ID ad14d79e419fb860ee59071d7c557810f913a4cc
>> # Parent  b5b6bd456279f011ec94f2a16a708d2833dd8119
>> debian: use a wildcard target for docker-debian-$codename
>> 
>> This makes it easier to build packages for stretch as well as jessie.
>> 
>> diff --git a/Makefile b/Makefile
>> --- a/Makefile
>> +++ b/Makefile
>> @@ -193,9 +193,11 @@ deb:
>>  ppa:
>>  	contrib/builddeb --source-only
>>  
>> -docker-debian-jessie:
>> -	mkdir -p packages/debian-jessie
>> -	contrib/dockerdeb debian jessie
>> +# Use the code name here, eg jessie for debian 8 and stretch for
>> +# debian 9
>> +docker-debian-%:
>> +	mkdir -p packages/debian-$*
>> +	contrib/dockerdeb debian $*
>
> But contrib/dockerdeb uses initcontainer() from contrib/dockerlib.sh,
> which needs a dockerfile in contrib/docker/ to work. For Jessie there
> is debian-jessie file, but for stretch there is nothing.
>
> I think we should adapt e63dfbbdbd07 for debian targets unless build
> dependencies for Jessie and Stretch are different.

Hmm, that's my commit ... oh, do you mean make a debian.template file as
I did for ubuntu? If so, then sure, that makes sense.

Might be nice before the release? I have no horse in this race, though.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 800 bytes
Desc: not available
URL: <http://www.mercurial-scm.org/pipermail/mercurial-devel/attachments/20170424/98e3402d/attachment.sig>


More information about the Mercurial-devel mailing list