All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [RFC 1/2] docs/manual/migrating.txt: add section with general migrating tips
@ 2021-08-02 16:22 Arnout Vandecappelle (Essensium/Mind)
  2021-08-02 16:22 ` [Buildroot] [RFC 2/2] docs/manual/migrating.txt: add "individual packages" section Arnout Vandecappelle (Essensium/Mind)
  2021-08-02 20:55 ` [Buildroot] [RFC 1/2] docs/manual/migrating.txt: add section with general migrating tips Yann E. MORIN
  0 siblings, 2 replies; 9+ messages in thread
From: Arnout Vandecappelle (Essensium/Mind) @ 2021-08-02 16:22 UTC (permalink / raw)
  To: buildroot; +Cc: Michael Nosthoff, Yann E . MORIN, Thomas Petazzoni

This is based on Arnout's experience with migrating Buildroot.

Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 docs/manual/migrating.txt | 28 ++++++++++++++++++++++++++++
 1 file changed, 28 insertions(+)

diff --git a/docs/manual/migrating.txt b/docs/manual/migrating.txt
index 92e487c71e..9fd7d7e676 100644
--- a/docs/manual/migrating.txt
+++ b/docs/manual/migrating.txt
@@ -8,6 +8,34 @@ Some versions have introduced backward incompatibilities. This section
 explains those incompatibilities, and for each explains what to do to
 complete the migration.
 
+[[migrating-approach]]
+=== General approach
+
+To migrate from an older Buildroot version, take the following steps.
+
+. For all your configurations, do a build in the old Buildroot
+  environment. Save the full +.config+ and
+  +build/packages-file-list.txt+.
+. Review the specific migration notes below and make the required
+  adaptations to external packages and custom build scripts.
+. In the new Buildroot environment, run +make menuconfig+ starting from
+  the existing +.config+.
+. If anything is enabled in the Legacy menu, check its help text,
+  unselect it, and save the configuration.
+. Review the CHANGES file to see if any of your packages and features
+  are affected by the changes.
+. Build in the new Buildroot environment.
+. Fix build issues in external packages (usually due to updated
+  dependencies).
+. Compare the new +packages-file-list.txt+ with the original one, to
+  check if no required files have disappeared.
+. For configuration (and other) files in a custom overlay that overwrite
+  files created by Buildroot, check if there are changes in the
+  Buildroot-generated file that need to be propagated to your custom
+  file.
+. Run +make savedefconfig+ and verify that what is selected really is
+  what you intended to enable.
+
 [[br2-external-converting]]
 === Migrating to 2016.11
 
-- 
2.31.1

_______________________________________________
buildroot mailing list
buildroot@busybox.net
http://lists.busybox.net/mailman/listinfo/buildroot

^ permalink raw reply related	[flat|nested] 9+ messages in thread

* [Buildroot] [RFC 2/2] docs/manual/migrating.txt: add "individual packages" section
  2021-08-02 16:22 [Buildroot] [RFC 1/2] docs/manual/migrating.txt: add section with general migrating tips Arnout Vandecappelle (Essensium/Mind)
@ 2021-08-02 16:22 ` Arnout Vandecappelle (Essensium/Mind)
  2021-08-03  5:48   ` Yann E. MORIN
  2021-08-02 20:55 ` [Buildroot] [RFC 1/2] docs/manual/migrating.txt: add section with general migrating tips Yann E. MORIN
  1 sibling, 1 reply; 9+ messages in thread
From: Arnout Vandecappelle (Essensium/Mind) @ 2021-08-02 16:22 UTC (permalink / raw)
  To: buildroot; +Cc: Michael Nosthoff, Yann E . MORIN, Thomas Petazzoni

Add a section to the migrating documentation to explain what changed in
individual packages and how it may affect your build.

This section is organised per package, not per Buildroot release,
because in general it's easier to check package per package if anything
changed than to check release per release for all your packages.

The idea is that every time a package is changed in a way that is not
immediately obvious (mainly: affects runtime but not build time), it is
mentioned in this section. This may include:

- a new version of the package got a new name due to API
  incompatibilities;
- the default behaviour or config optoins of the package changed;
- the Buildroot integration (e.g. configuration file, init script) of
  the package changed;
- what is installed by the package is changed (e.g. executable is
  renamed or removed).

To start with, add the recent changes of the installation defaults of
bluez5_utils.

Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 docs/manual/migrating.txt | 15 +++++++++++++--
 1 file changed, 13 insertions(+), 2 deletions(-)

diff --git a/docs/manual/migrating.txt b/docs/manual/migrating.txt
index 9fd7d7e676..36e4aa4724 100644
--- a/docs/manual/migrating.txt
+++ b/docs/manual/migrating.txt
@@ -22,8 +22,8 @@ To migrate from an older Buildroot version, take the following steps.
   the existing +.config+.
 . If anything is enabled in the Legacy menu, check its help text,
   unselect it, and save the configuration.
-. Review the CHANGES file to see if any of your packages and features
-  are affected by the changes.
+. Review the individual packages section below to see if any of your
+  packages are affected by the changes.
 . Build in the new Buildroot environment.
 . Fix build issues in external packages (usually due to updated
   dependencies).
@@ -36,6 +36,17 @@ To migrate from an older Buildroot version, take the following steps.
 . Run +make savedefconfig+ and verify that what is selected really is
   what you intended to enable.
 
+[[migrating-packages]]
+=== Individual packages
+
+This section describes changes to individual packages. For all packages
+that are built in your configuration, check below if changes are needed.
+
+==== bluez5_utils
+
+Since 2021.08, the tools and all plugins are optional and default off.
+Make sure to enable the plugins you require.
+
 [[br2-external-converting]]
 === Migrating to 2016.11
 
-- 
2.31.1

_______________________________________________
buildroot mailing list
buildroot@busybox.net
http://lists.busybox.net/mailman/listinfo/buildroot

^ permalink raw reply related	[flat|nested] 9+ messages in thread

* Re: [Buildroot] [RFC 1/2] docs/manual/migrating.txt: add section with general migrating tips
  2021-08-02 16:22 [Buildroot] [RFC 1/2] docs/manual/migrating.txt: add section with general migrating tips Arnout Vandecappelle (Essensium/Mind)
  2021-08-02 16:22 ` [Buildroot] [RFC 2/2] docs/manual/migrating.txt: add "individual packages" section Arnout Vandecappelle (Essensium/Mind)
@ 2021-08-02 20:55 ` Yann E. MORIN
  2021-08-03 15:18   ` Arnout Vandecappelle
  1 sibling, 1 reply; 9+ messages in thread
From: Yann E. MORIN @ 2021-08-02 20:55 UTC (permalink / raw)
  To: Arnout Vandecappelle (Essensium/Mind)
  Cc: Michael Nosthoff, Thomas Petazzoni, buildroot

Arnout, All,

On 2021-08-02 18:22 +0200, Arnout Vandecappelle (Essensium/Mind) spake thusly:
> This is based on Arnout's experience with migrating Buildroot.

Except for the details, this also quite closely matches my own
experience.

> Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
> ---
>  docs/manual/migrating.txt | 28 ++++++++++++++++++++++++++++
>  1 file changed, 28 insertions(+)
> 
> diff --git a/docs/manual/migrating.txt b/docs/manual/migrating.txt
> index 92e487c71e..9fd7d7e676 100644
> --- a/docs/manual/migrating.txt
> +++ b/docs/manual/migrating.txt
> @@ -8,6 +8,34 @@ Some versions have introduced backward incompatibilities. This section
>  explains those incompatibilities, and for each explains what to do to
>  complete the migration.
>  
> +[[migrating-approach]]
> +=== General approach
> +
> +To migrate from an older Buildroot version, take the following steps.
> +
> +. For all your configurations, do a build in the old Buildroot
> +  environment. Save the full +.config+ and
> +  +build/packages-file-list.txt+.

(read my comment below first, then come back here)
Maybe run 'make clean' here?

> +. Review the specific migration notes below and make the required
> +  adaptations to external packages and custom build scripts.
> +. In the new Buildroot environment, run +make menuconfig+ starting from
> +  the existing +.config+.
> +. If anything is enabled in the Legacy menu, check its help text,
> +  unselect it, and save the configuration.
> +. Review the CHANGES file to see if any of your packages and features
> +  are affected by the changes.
> +. Build in the new Buildroot environment.

Above, you said to start with a full build, so if folowing this sequence
to the letter, we still have a full build. We need to run 'make clean'
first (go back reading my comment, above).

> +. Fix build issues in external packages (usually due to updated
> +  dependencies).
> +. Compare the new +packages-file-list.txt+ with the original one, to
> +  check if no required files have disappeared.

Maybe also suggest graph-size et al. about investigating size changes,
too?

> +. For configuration (and other) files in a custom overlay that overwrite
> +  files created by Buildroot, check if there are changes in the
> +  Buildroot-generated file that need to be propagated to your custom
> +  file.
> +. Run +make savedefconfig+ and verify that what is selected really is
> +  what you intended to enable.

I am always wary instructing people to look at defconfig files; I always
consider them to be artefacts of the configuration.

Rather, I suggest people read the output of legal-info to check that all
the packages there are acceptable, and see graph-depends to see how a
package came to the build.

Regards,
Yann E. MORIN.

>  [[br2-external-converting]]
>  === Migrating to 2016.11
>  
> -- 
> 2.31.1
> 

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@busybox.net
http://lists.busybox.net/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [Buildroot] [RFC 2/2] docs/manual/migrating.txt: add "individual packages" section
  2021-08-02 16:22 ` [Buildroot] [RFC 2/2] docs/manual/migrating.txt: add "individual packages" section Arnout Vandecappelle (Essensium/Mind)
@ 2021-08-03  5:48   ` Yann E. MORIN
  2021-08-03 15:19     ` Arnout Vandecappelle
  0 siblings, 1 reply; 9+ messages in thread
From: Yann E. MORIN @ 2021-08-03  5:48 UTC (permalink / raw)
  To: Arnout Vandecappelle (Essensium/Mind)
  Cc: Michael Nosthoff, Thomas Petazzoni, buildroot

Arnout, All,

On 2021-08-02 18:22 +0200, Arnout Vandecappelle (Essensium/Mind) spake thusly:
> Add a section to the migrating documentation to explain what changed in
> individual packages and how it may affect your build.
> 
> This section is organised per package, not per Buildroot release,
> because in general it's easier to check package per package if anything
> changed than to check release per release for all your packages.
> 
> The idea is that every time a package is changed in a way that is not
> immediately obvious (mainly: affects runtime but not build time), it is
> mentioned in this section. This may include:
> 
> - a new version of the package got a new name due to API
>   incompatibilities;
> - the default behaviour or config optoins of the package changed;
> - the Buildroot integration (e.g. configuration file, init script) of
>   the package changed;
> - what is installed by the package is changed (e.g. executable is
>   renamed or removed).

No. Please, no.

We already have two locations were changes in packages are identified:

  - git commit logs, the authoritative reference,

  - the CHANGES files

I do not want to have to maintain yet a third, ever-growing and
never-trimmed list. Also, we will always have to bikeshed whther such a
change is disruptive enough or not to warrant being on that list...

Instead, I would just suggest that people run "git log --oneline" or
"git log -p --stat" on the packages they are interested in.

And of course, that means we *need* well-written commit logs (but we're
already pretty good I believe).

Having a section about the core internals, like the br2-defconfig stuff
that is not obvious, or about the RPATH of host packages, is acceptable,
because it is not easily discoverable.

Regards,
Yann E. MORIN.

> To start with, add the recent changes of the installation defaults of
> bluez5_utils.
> 
> Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
> ---
>  docs/manual/migrating.txt | 15 +++++++++++++--
>  1 file changed, 13 insertions(+), 2 deletions(-)
> 
> diff --git a/docs/manual/migrating.txt b/docs/manual/migrating.txt
> index 9fd7d7e676..36e4aa4724 100644
> --- a/docs/manual/migrating.txt
> +++ b/docs/manual/migrating.txt
> @@ -22,8 +22,8 @@ To migrate from an older Buildroot version, take the following steps.
>    the existing +.config+.
>  . If anything is enabled in the Legacy menu, check its help text,
>    unselect it, and save the configuration.
> -. Review the CHANGES file to see if any of your packages and features
> -  are affected by the changes.
> +. Review the individual packages section below to see if any of your
> +  packages are affected by the changes.
>  . Build in the new Buildroot environment.
>  . Fix build issues in external packages (usually due to updated
>    dependencies).
> @@ -36,6 +36,17 @@ To migrate from an older Buildroot version, take the following steps.
>  . Run +make savedefconfig+ and verify that what is selected really is
>    what you intended to enable.
>  
> +[[migrating-packages]]
> +=== Individual packages
> +
> +This section describes changes to individual packages. For all packages
> +that are built in your configuration, check below if changes are needed.
> +
> +==== bluez5_utils
> +
> +Since 2021.08, the tools and all plugins are optional and default off.
> +Make sure to enable the plugins you require.
> +
>  [[br2-external-converting]]
>  === Migrating to 2016.11
>  
> -- 
> 2.31.1
> 

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@busybox.net
http://lists.busybox.net/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [Buildroot] [RFC 1/2] docs/manual/migrating.txt: add section with general migrating tips
  2021-08-02 20:55 ` [Buildroot] [RFC 1/2] docs/manual/migrating.txt: add section with general migrating tips Yann E. MORIN
@ 2021-08-03 15:18   ` Arnout Vandecappelle
  0 siblings, 0 replies; 9+ messages in thread
From: Arnout Vandecappelle @ 2021-08-03 15:18 UTC (permalink / raw)
  To: Yann E. MORIN; +Cc: Michael Nosthoff, Thomas Petazzoni, buildroot



On 02/08/2021 22:55, Yann E. MORIN wrote:
> Arnout, All,
> 
> On 2021-08-02 18:22 +0200, Arnout Vandecappelle (Essensium/Mind) spake thusly:
>> This is based on Arnout's experience with migrating Buildroot.
> 
> Except for the details, this also quite closely matches my own
> experience.

 Excellent. I'll remove the RFC bit from v2 then.

> 
>> Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
>> ---
>>  docs/manual/migrating.txt | 28 ++++++++++++++++++++++++++++
>>  1 file changed, 28 insertions(+)
>>
>> diff --git a/docs/manual/migrating.txt b/docs/manual/migrating.txt
>> index 92e487c71e..9fd7d7e676 100644
>> --- a/docs/manual/migrating.txt
>> +++ b/docs/manual/migrating.txt
>> @@ -8,6 +8,34 @@ Some versions have introduced backward incompatibilities. This section
>>  explains those incompatibilities, and for each explains what to do to
>>  complete the migration.
>>  
>> +[[migrating-approach]]
>> +=== General approach
>> +
>> +To migrate from an older Buildroot version, take the following steps.
>> +
>> +. For all your configurations, do a build in the old Buildroot
>> +  environment. Save the full +.config+ and
>> +  +build/packages-file-list.txt+.
> 
> (read my comment below first, then come back here)
> Maybe run 'make clean' here?

 Mentally I assumed there would be an "old" buildroot directory with its output
directory, and a "new" buildroot directory with its distinct output directory.
But I indeed didn't write that, and in fact I don't work like that myself. So
'make clean' it is!


>> +. Review the specific migration notes below and make the required
>> +  adaptations to external packages and custom build scripts.
>> +. In the new Buildroot environment, run +make menuconfig+ starting from
>> +  the existing +.config+.
>> +. If anything is enabled in the Legacy menu, check its help text,
>> +  unselect it, and save the configuration.
>> +. Review the CHANGES file to see if any of your packages and features
>> +  are affected by the changes.
>> +. Build in the new Buildroot environment.
> 
> Above, you said to start with a full build, so if folowing this sequence
> to the letter, we still have a full build. We need to run 'make clean'
> first (go back reading my comment, above).
> 
>> +. Fix build issues in external packages (usually due to updated
>> +  dependencies).
>> +. Compare the new +packages-file-list.txt+ with the original one, to
>> +  check if no required files have disappeared.
> 
> Maybe also suggest graph-size et al. about investigating size changes,
> too?

 Excellent idea. You can easily compare the csv files.


>> +. For configuration (and other) files in a custom overlay that overwrite
>> +  files created by Buildroot, check if there are changes in the
>> +  Buildroot-generated file that need to be propagated to your custom
>> +  file.
>> +. Run +make savedefconfig+ and verify that what is selected really is
>> +  what you intended to enable.
> 
> I am always wary instructing people to look at defconfig files; I always
> consider them to be artefacts of the configuration.

 I agree with the principle. Unfortunately, I know of no better way of getting
the "top-level" packages.


> Rather, I suggest people read the output of legal-info to check that all
> the packages there are acceptable, and see graph-depends to see how a
> package came to the build.

 Theoretically, graph-depends gives the top-level packages as well (i.e. the
packages that ALL points to). However, for a reasonably-sized project with about
50 top-level packages, the graph is pretty much unreadable. 'grep -w _all
graphs/graph-depends.dot' is an alternative but it's not exactly user-friendly -
and it doesn't show the selected sub-options like the defconfig does.

 The defconfig isn't great either, of course. For example, it doesn't show the
sub-options that default y, and also choices aren't expressed very clearly.

 Actually, the main reason I use the defconfig is that it comes for free: 'make
savedefconfig; git diff'. So maybe I should mention that instead.

 Since this is a bit controversial, however, I'll just drop this point.


 Regarding legal-info: if it's just about getting a list of packages, graph-size
already gives that, so I think I will talk about graph-size but not about
legal-info.


 Regards,
 Arnout



> 
> Regards,
> Yann E. MORIN.
> 
>>  [[br2-external-converting]]
>>  === Migrating to 2016.11
>>  
>> -- 
>> 2.31.1
>>
> 
_______________________________________________
buildroot mailing list
buildroot@busybox.net
http://lists.busybox.net/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [Buildroot] [RFC 2/2] docs/manual/migrating.txt: add "individual packages" section
  2021-08-03  5:48   ` Yann E. MORIN
@ 2021-08-03 15:19     ` Arnout Vandecappelle
  2021-08-03 18:56       ` Michael Nosthoff via buildroot
  0 siblings, 1 reply; 9+ messages in thread
From: Arnout Vandecappelle @ 2021-08-03 15:19 UTC (permalink / raw)
  To: Yann E. MORIN; +Cc: Michael Nosthoff, Thomas Petazzoni, buildroot



On 03/08/2021 07:48, Yann E. MORIN wrote:
> Arnout, All,
> 
> On 2021-08-02 18:22 +0200, Arnout Vandecappelle (Essensium/Mind) spake thusly:
>> Add a section to the migrating documentation to explain what changed in
>> individual packages and how it may affect your build.
>>
>> This section is organised per package, not per Buildroot release,
>> because in general it's easier to check package per package if anything
>> changed than to check release per release for all your packages.
>>
>> The idea is that every time a package is changed in a way that is not
>> immediately obvious (mainly: affects runtime but not build time), it is
>> mentioned in this section. This may include:
>>
>> - a new version of the package got a new name due to API
>>   incompatibilities;
>> - the default behaviour or config optoins of the package changed;
>> - the Buildroot integration (e.g. configuration file, init script) of
>>   the package changed;
>> - what is installed by the package is changed (e.g. executable is
>>   renamed or removed).
> 
> No. Please, no.
> 
> We already have two locations were changes in packages are identified:
> 
>   - git commit logs, the authoritative reference,
> 
>   - the CHANGES files
> 
> I do not want to have to maintain yet a third, ever-growing and
> never-trimmed list. Also, we will always have to bikeshed whther such a
> change is disruptive enough or not to warrant being on that list...
> 
> Instead, I would just suggest that people run "git log --oneline" or
> "git log -p --stat" on the packages they are interested in.

 That's good advice as well. Simple "git log" should be sufficient though, if
the commit log is OK. "git log --oneline" is definitely not OK, because
important changes during a version bump will not appear in the oneline.

 I'm going to add a bullet point for this and merge it into the first patch.

 Regards,
 Arnout

> 
> And of course, that means we *need* well-written commit logs (but we're
> already pretty good I believe).
> 
> Having a section about the core internals, like the br2-defconfig stuff
> that is not obvious, or about the RPATH of host packages, is acceptable,
> because it is not easily discoverable.
> 
> Regards,
> Yann E. MORIN.
> 
>> To start with, add the recent changes of the installation defaults of
>> bluez5_utils.
>>
>> Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
>> ---
>>  docs/manual/migrating.txt | 15 +++++++++++++--
>>  1 file changed, 13 insertions(+), 2 deletions(-)
>>
>> diff --git a/docs/manual/migrating.txt b/docs/manual/migrating.txt
>> index 9fd7d7e676..36e4aa4724 100644
>> --- a/docs/manual/migrating.txt
>> +++ b/docs/manual/migrating.txt
>> @@ -22,8 +22,8 @@ To migrate from an older Buildroot version, take the following steps.
>>    the existing +.config+.
>>  . If anything is enabled in the Legacy menu, check its help text,
>>    unselect it, and save the configuration.
>> -. Review the CHANGES file to see if any of your packages and features
>> -  are affected by the changes.
>> +. Review the individual packages section below to see if any of your
>> +  packages are affected by the changes.
>>  . Build in the new Buildroot environment.
>>  . Fix build issues in external packages (usually due to updated
>>    dependencies).
>> @@ -36,6 +36,17 @@ To migrate from an older Buildroot version, take the following steps.
>>  . Run +make savedefconfig+ and verify that what is selected really is
>>    what you intended to enable.
>>  
>> +[[migrating-packages]]
>> +=== Individual packages
>> +
>> +This section describes changes to individual packages. For all packages
>> +that are built in your configuration, check below if changes are needed.
>> +
>> +==== bluez5_utils
>> +
>> +Since 2021.08, the tools and all plugins are optional and default off.
>> +Make sure to enable the plugins you require.
>> +
>>  [[br2-external-converting]]
>>  === Migrating to 2016.11
>>  
>> -- 
>> 2.31.1
>>
> 
_______________________________________________
buildroot mailing list
buildroot@busybox.net
http://lists.busybox.net/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [Buildroot] [RFC 2/2] docs/manual/migrating.txt: add "individual packages" section
  2021-08-03 15:19     ` Arnout Vandecappelle
@ 2021-08-03 18:56       ` Michael Nosthoff via buildroot
  2021-08-03 19:47         ` Yann E. MORIN
  0 siblings, 1 reply; 9+ messages in thread
From: Michael Nosthoff via buildroot @ 2021-08-03 18:56 UTC (permalink / raw)
  To: Arnout Vandecappelle, Yann E. MORIN; +Cc: Thomas Petazzoni, buildroot

Hi,


On 03.08.21 17:19, Arnout Vandecappelle wrote:
>
> On 03/08/2021 07:48, Yann E. MORIN wrote:
>> Arnout, All,
>>
>> On 2021-08-02 18:22 +0200, Arnout Vandecappelle (Essensium/Mind) spake thusly:
>>> Add a section to the migrating documentation to explain what changed in
>>> individual packages and how it may affect your build.
>>>
>>> This section is organised per package, not per Buildroot release,
>>> because in general it's easier to check package per package if anything
>>> changed than to check release per release for all your packages.
>>>
>>> The idea is that every time a package is changed in a way that is not
>>> immediately obvious (mainly: affects runtime but not build time), it is
>>> mentioned in this section. This may include:
>>>
>>> - a new version of the package got a new name due to API
>>>    incompatibilities;
>>> - the default behaviour or config optoins of the package changed;
>>> - the Buildroot integration (e.g. configuration file, init script) of
>>>    the package changed;
>>> - what is installed by the package is changed (e.g. executable is
>>>    renamed or removed).
>> No. Please, no.
>>
>> We already have two locations were changes in packages are identified:
>>
>>    - git commit logs, the authoritative reference,
>>
>>    - the CHANGES files
>>
>> I do not want to have to maintain yet a third, ever-growing and
>> never-trimmed list. Also, we will always have to bikeshed whther such a
>> change is disruptive enough or not to warrant being on that list...
>>
>> Instead, I would just suggest that people run "git log --oneline" or
>> "git log -p --stat" on the packages they are interested in.
>   That's good advice as well. Simple "git log" should be sufficient though, if
> the commit log is OK. "git log --oneline" is definitely not OK, because
> important changes during a version bump will not appear in the oneline.
>
>   I'm going to add a bullet point for this and merge it into the first patch.

One thing I want to point out: The releases are tarballs. Should we 
assume the user has the git history available?


Regards,
Michael

_______________________________________________
buildroot mailing list
buildroot@busybox.net
http://lists.busybox.net/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [Buildroot] [RFC 2/2] docs/manual/migrating.txt: add "individual packages" section
  2021-08-03 18:56       ` Michael Nosthoff via buildroot
@ 2021-08-03 19:47         ` Yann E. MORIN
  2021-08-03 21:35           ` Thomas Petazzoni
  0 siblings, 1 reply; 9+ messages in thread
From: Yann E. MORIN @ 2021-08-03 19:47 UTC (permalink / raw)
  To: Michael Nosthoff; +Cc: Thomas Petazzoni, buildroot

Micheal, All,

On 2021-08-03 20:56 +0200, Michael Nosthoff spake thusly:
> On 03.08.21 17:19, Arnout Vandecappelle wrote:
> >On 03/08/2021 07:48, Yann E. MORIN wrote:
> >>On 2021-08-02 18:22 +0200, Arnout Vandecappelle (Essensium/Mind) spake thusly:
> >>>Add a section to the migrating documentation to explain what changed in
> >>>individual packages and how it may affect your build.
[--SNIP--]
> >>Instead, I would just suggest that people run "git log --oneline" or
> >>"git log -p --stat" on the packages they are interested in.
> >  That's good advice as well. Simple "git log" should be sufficient though, if
> >the commit log is OK. "git log --oneline" is definitely not OK, because
> >important changes during a version bump will not appear in the oneline.
> >
> >  I'm going to add a bullet point for this and merge it into the first patch.
> 
> One thing I want to point out: The releases are tarballs. Should we assume
> the user has the git history available?

People are really still using tarballs in these days? ;-) But OK, you've
got a point.

However, I would still think that people who want to follow a process
when updating are probably already using buildroot a git submodule of
some sorts, so yes, they will have a git tree to look at.

People who use a released tarball are not really looking for a sustained
or long-term vision, and are not planning on upgrading.

So, I think we can state something like:

  - look at the CHANGES file, and run 'git log' on the packages of your
    interest; if you are not using git, clone the upstream repository
    for this.

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@busybox.net
http://lists.busybox.net/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [Buildroot] [RFC 2/2] docs/manual/migrating.txt: add "individual packages" section
  2021-08-03 19:47         ` Yann E. MORIN
@ 2021-08-03 21:35           ` Thomas Petazzoni
  0 siblings, 0 replies; 9+ messages in thread
From: Thomas Petazzoni @ 2021-08-03 21:35 UTC (permalink / raw)
  To: Yann E. MORIN; +Cc: Michael Nosthoff, buildroot

Hello,

On Tue, 3 Aug 2021 21:47:58 +0200
"Yann E. MORIN" <yann.morin.1998@free.fr> wrote:

> People are really still using tarballs in these days? ;-) But OK, you've
> got a point.
> 
> However, I would still think that people who want to follow a process
> when updating are probably already using buildroot a git submodule of
> some sorts, so yes, they will have a git tree to look at.
> 
> People who use a released tarball are not really looking for a sustained
> or long-term vision, and are not planning on upgrading.

I was also surprised, but the other day, a person working for a large
IP vendor company complained that he was not able to download the
Buildroot tarballs. For some reason, the IP of this company had been
blocked in iptables rules of buildroot.org. When I suggested to use Git
instead, and especially the Github or Gitlab mirrors as a workaround,
it apparently wasn't considered really a good solution.

So it does seem like some people are still using the tarballs, indeed.

Thomas
-- 
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@busybox.net
http://lists.busybox.net/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2021-08-04  1:19 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-08-02 16:22 [Buildroot] [RFC 1/2] docs/manual/migrating.txt: add section with general migrating tips Arnout Vandecappelle (Essensium/Mind)
2021-08-02 16:22 ` [Buildroot] [RFC 2/2] docs/manual/migrating.txt: add "individual packages" section Arnout Vandecappelle (Essensium/Mind)
2021-08-03  5:48   ` Yann E. MORIN
2021-08-03 15:19     ` Arnout Vandecappelle
2021-08-03 18:56       ` Michael Nosthoff via buildroot
2021-08-03 19:47         ` Yann E. MORIN
2021-08-03 21:35           ` Thomas Petazzoni
2021-08-02 20:55 ` [Buildroot] [RFC 1/2] docs/manual/migrating.txt: add section with general migrating tips Yann E. MORIN
2021-08-03 15:18   ` Arnout Vandecappelle

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.