Summary of inadvertently private mails:
mk-bsd-sed-quotation-mark.diff hopefully fixes the build on BSD.
Assuming this, and that I am not missing the point again,
mk-unhardcode-datadir.diff can be applied again.
The attachment improves its readability.
* It replaces "ifndef ... := ... endif" wit
Hi!
On Mon, 2024-07-01 at 04:43:40 +0200, Guillem Jover wrote:
> On Sat, 2024-06-29 at 17:29:49 +0200, Guillem Jover wrote:
> > There's another regression that I found on various BSDs, where they
> > already have GNU make 4.4.1, where the buildtools test fails for the
> > AR and AR_FOR_BUILD check
Hi!
On Sat, 2024-06-29 at 17:29:49 +0200, Guillem Jover wrote:
> There's another regression that I found on various BSDs, where they
> already have GNU make 4.4.1, where the buildtools test fails for the
> AR and AR_FOR_BUILD checks, and I've not managed to track down yet
> what the problem is (an
Hi!
On Mon, 2024-05-06 at 12:32:45 +0200, Nicolas Boulenguez wrote:
> You have not applied
> 0001-scripts-mk-stop-hard-coding-dpkg_datadir.patch
> probably because you prefer the related parts in
> f1175056 (build: Rework subst handling for built or installed artifacts).
As I think I mentione
Package: dpkg-dev
Followup-For: Bug #872381
Hello again.
My last message was confusing.
I am suggesting to improve commit f1175056 with
0001-build-spare-an-unneeded-subst-handling-in-pkg-info.m.patch.
I have only quoted 0001-scripts-mk-stop-hard-coding-dpkg_datadir.patch
for context. Please igno
Package: dpkg-dev
Followup-For: Bug #872381
Hello.
It is good to see the main suggestion merged. Thanks!
You have not applied
0001-scripts-mk-stop-hard-coding-dpkg_datadir.patch
probably because you prefer the related parts in
f1175056 (build: Rework subst handling for built or installed arti
The attached version
fixes the spacing issue in tests with a more readable trick,
splits the changes inside tests into small chunks,
replaces - with _ in Make variable names,
avoids non portable -r -E sed options.
Without -r, sed in {buildflags,pkg-info}.mk should cause no
regression. It has been
Hi!
On Mon, 2024-03-11 at 01:44:30 +0100, Nicolas Boulenguez wrote:
> Package: dpkg-dev
> Followup-For: Bug #872381
> Please consider this new patch queue instead of the old or untested
> ones. With this one applied on 279c6ccb, the package builds and
> passes all tests.
Thanks!
> * scripts/mk
Package: dpkg-dev
Followup-For: Bug #872381
Hello.
Please consider this new patch queue instead of the old or untested
ones. With this one applied on 279c6ccb, the package builds and
passes all tests.
* scripts/mk: only use ASCII characters
Cosmetic independent suggestion.
* scripts/mk: prot
Package: dpkg-dev
Followup-For: Bug #872381
This new version, based on c881a5a8,
* splits protection from double inclusion and dpkg_datadir generation
into separate commits
* fixes an error in DEB_BUILD_OPTION_PARALLEL
* removes a few dubious optimizations (like checking if dpkg_datadir
is alr
Package: dpkg-dev
Followup-For: Bug #872381
Hello.
The attached commits rebase the suggestions, take your answers into
account and slightly improved some style issues.
There may remain typos, nothing is tested this time.
>From d56d5af7fa1a01a581d0cc1901572ca9c407f538 Mon Sep 17 00:00:00 2001
From:
-- stop hard-coding dpkg_datadir
> to avoid changing all pathname concatenation I changed dpkg_datadir to
> «$(patsubst %/,%,$(dir $(lastword $(MAKEFILE_LIST». But given that
> I think we might still need to substitute other things I've left this
> one out for now.
A rebased and slightly less
Hi!
Thanks for the updated patches! I've gone over these and merged
several of them, the others I've left out for now, as I wanted to
get the current release out, which has been dragging for too long
already while I was finalizing it.
On Sun, 2022-02-13 at 18:38:19 +0100, Nicolas Boulenguez wrote
Package: dpkg-dev
Followup-For: Bug #872381
Of course,
-dpkg_datadir := $(shell $(dir $(lastword $(MAKEFILE_LIST
+dpkg_datadir := $(dir $(lastword $(MAKEFILE_LIST)))
Hello.
I have rebased the changes and taken your explanations into
account. This new patch queue is not tested again, but shoul
> > [in scripts/mk/Makefile.am], I suggest to rename
> > scripts/mk/{default,buildtools}.mk to scripts/mk/*.mk.sed or similar
> > (for example, .sed.mk in order to keep syntax highlighting). Distinct
> > source and object files would also simplify scripts/mk/Makefile a lot.
> Yeah, that would be
Hi!
On Thu, 2022-02-10 at 21:53:59 +0100, Nicolas Boulenguez wrote:
> Package: dpkg-dev
> Followup-For: Bug #872381
> The --in-place sed option ([1]) may simplify scripts/mk/Makefile.am
> much more than the do_make_subst variable provided by
> build-aux/subst.am does. It is for now the only Makef
Package: dpkg-dev
Followup-For: Bug #872381
(CCing Guillem as the author of the commits referenced below)
Hello.
The changes introduced by 1947fef8 and 981d5acb are basically
incompatible with the first patch in bug report #872381 [1].
For the record, commits a49f0a9c and f28cbcc4 also require
Package: dpkg-dev
Followup-For: Bug #872381
One more suggestion is attached.
>From 95a908b270a533fc5d4a58a45deb929ce0d6159e Mon Sep 17 00:00:00 2001
From: Nicolas Boulenguez
Date: Mon, 1 Nov 2021 10:08:08 +0100
Subject: [PATCH 8/8] scripts/mk/buildopts.mk: small optimisations
Assign DEB_BUILD_OP
Package: dpkg-dev
Version: 1.20.9
Followup-For: Bug #872381
The attached commits are, once more, manually rebased on the current
HEAD.
>From ad7a566e7966f7c8062a044d0c2df0b07b0011d7 Mon Sep 17 00:00:00 2001
From: Nicolas Boulenguez
Date: Mon, 29 Jul 2019 14:38:32 +0200
Subject: [PATCH 1/8] script
Package: dpkg-dev
Followup-For: Bug #872381
The attached changes are split into smaller chunks and rebased on
55cdc52e. A merge conflict with 52166568 could have easily been
avoided, please review pending patches before reformatting comments.
>From f021e4d5ada4df39948afa339bd3a0859d28ed5a Mon Sep
Please replace 0003.patch with the attached version (changes are
cosmetic but should use review and merge).
>From a93a87581dbe90bd15822f11f374ae6d76263ca4 Mon Sep 17 00:00:00 2001
From: Nicolas Boulenguez
Date: Mon, 29 Jul 2019 19:03:16 +0200
Subject: [PATCH 3 (v2)/3] scripts/mk: improve subproces
Package: dpkg-dev
Version: 1.19.7
Followup-For: Bug #872381
Hello.
The attachments are commits based on the current git head (f0c3cccf).
2/ adds tests for the Make snippets.
The current implementation passes these new tests.
3/ implements the suggestion as a git commit.
It also passes all tests
Package: dpkg-dev
Version: 1.19.2
Followup-For: Bug #872381
Hi.
This new version improves the readability in various aspects,
and should be easyer to review.
It fixes a difference with the current version in dpkg-dev by
exporting all dpkg-architecture variables.
Is there anything that I can do to
Package: dpkg-dev
Version: 1.18.24
Severity: wishlist
Tags: patch
Hello.
A new implementation of
/usr/share/dpkg/architecture.mk
/usr/share/dpkg/buildflags.mk
/usr/share/dpkg/pkg-info.mk
is attached. You may use run_tests.sh to check that the behaviour
remains the same in most circumstance
24 matches
Mail list logo
