From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Petazzoni Date: Wed, 15 Aug 2018 19:35:28 +0200 Subject: [Buildroot] [PATCH 3/3] New -update-last-config-fragment target in pkg-kconfig.mk In-Reply-To: <20180815161657.GC2556@scaer> References: <20180730155153.24091-1-m.patzlaff@pilz.de> <20180730155153.24091-4-m.patzlaff@pilz.de> <20180730234643.34315d11@windsurf> <20180731154948.GB8537@scaer> <20180814162705.505476fe@windsurf> <83afeab5-baf8-122e-15ab-17889c3a300d@mind.be> <20180815140413.62984968@windsurf> <20180815161657.GC2556@scaer> Message-ID: <20180815193528.6fc47d7b@windsurf> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: buildroot@busybox.net Hello, On Wed, 15 Aug 2018 18:16:57 +0200, Yann E. MORIN wrote: > Indeed, it is not to be considered to be a diff of any of the fragment, > but of the configuration *as a whole*. > > That's why I initially did not write a full diff, but only the delta > snippet, not the context, because it is a diff, not a patch. > > But I think we should instead use the scripts/diffconfig script that is > in the linux kernel tree, and which outputs looks like: > > $ cat config.000 > CONFIG_FOO=y > $ cat config.001 > CONFIG_FOO is not set > $ ./scripts/diffconfig config.000 config.001 > FOO y -> n > > which IMNSHO is exactly the message we want to convey. Indeed, this looks very good, and diffconfig is an existing tool. Thomas -- Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons) Embedded Linux and Kernel engineering https://bootlin.com