diff mbox series

Update Ubuntu essentials list, replace liblz4-tool with lz4

Message ID 6f93430f-7953-39b6-caf7-b144f37ab657@crashcourse.ca
State New
Headers show
Series Update Ubuntu essentials list, replace liblz4-tool with lz4 | expand

Commit Message

Robert P. J. Day June 2, 2025, 1:02 p.m. UTC
Update the "Ubuntu Essentials package list" by replacing the obsolete
liblz4-tool package name with the current lz4 package name.

According to Quentin Schulz:

"According to pkgs.org, 20.04 lz4 replaces/provides liblz4-tool, same
for 22.04 and 24.04 and for some reason, liblz4-tool requires lz4
package, so I think it should be fine to replace it yes.

"Debian seems to have the same packaging in Bookworm and Bullseye, so
probably fine to do the change. They only seem to be adding a
changelog and copyright, and require lz4 package in any case."

Signed-off-by: Robert P. J. Day <rpjday@crashcourse.ca>

---

Comments

Richard Purdie June 2, 2025, 1:07 p.m. UTC | #1
On Mon, 2025-06-02 at 09:02 -0400, Robert P. J. Day via lists.yoctoproject.org wrote:
> 
> Update the "Ubuntu Essentials package list" by replacing the obsolete
> liblz4-tool package name with the current lz4 package name.
> 
> According to Quentin Schulz:
> 
> "According to pkgs.org, 20.04 lz4 replaces/provides liblz4-tool, same
> for 22.04 and 24.04 and for some reason, liblz4-tool requires lz4
> package, so I think it should be fine to replace it yes.
> 
> "Debian seems to have the same packaging in Bookworm and Bullseye, so
> probably fine to do the change. They only seem to be adding a
> changelog and copyright, and require lz4 package in any case."
> 
> Signed-off-by: Robert P. J. Day <rpjday@crashcourse.ca>
> 
> ---
> 
> diff --git a/documentation/tools/host_packages_scripts/ubuntu_essential.sh b/documentation/tools/host_packages_scripts/ubuntu_essential.sh
> index f3388c88c..c4231188b 100644
> --- a/documentation/tools/host_packages_scripts/ubuntu_essential.sh
> +++ b/documentation/tools/host_packages_scripts/ubuntu_essential.sh
> @@ -1 +1 @@
> -sudo apt-get install build-essential chrpath cpio debianutils diffstat file gawk gcc git iputils-ping libacl1 liblz4-tool locales python3 python3-git python3-jinja2 python3-pexpect python3-pip python3-subunit socat texinfo unzip wget xz-utils zstd
> +sudo apt-get install build-essential chrpath cpio debianutils diffstat file gawk gcc git iputils-ping libacl1 lz4 locales python3 python3-git python3-jinja2 python3-pexpect python3-pip python3-subunit socat texinfo unzip wget xz-utils zstd
> 

Do we actually need lz4 on the system to run a build? This seems a
little strange to me...

Cheers,

RIchard
Robert P. J. Day June 2, 2025, 1:17 p.m. UTC | #2
On Mon, 2 Jun 2025, Richard Purdie via lists.yoctoproject.org wrote:

> On Mon, 2025-06-02 at 09:02 -0400, Robert P. J. Day via lists.yoctoproject.org wrote:
> >
> > Update the "Ubuntu Essentials package list" by replacing the obsolete
> > liblz4-tool package name with the current lz4 package name.
> >
> > According to Quentin Schulz:
> >
> > "According to pkgs.org, 20.04 lz4 replaces/provides liblz4-tool, same
> > for 22.04 and 24.04 and for some reason, liblz4-tool requires lz4
> > package, so I think it should be fine to replace it yes.
> >
> > "Debian seems to have the same packaging in Bookworm and Bullseye, so
> > probably fine to do the change. They only seem to be adding a
> > changelog and copyright, and require lz4 package in any case."
> >
> > Signed-off-by: Robert P. J. Day <rpjday@crashcourse.ca>
> >
> > ---
> >
> > diff --git a/documentation/tools/host_packages_scripts/ubuntu_essential.sh b/documentation/tools/host_packages_scripts/ubuntu_essential.sh
> > index f3388c88c..c4231188b 100644
> > --- a/documentation/tools/host_packages_scripts/ubuntu_essential.sh
> > +++ b/documentation/tools/host_packages_scripts/ubuntu_essential.sh
> > @@ -1 +1 @@
> > -sudo apt-get install build-essential chrpath cpio debianutils diffstat file gawk gcc git iputils-ping libacl1 liblz4-tool locales python3 python3-git python3-jinja2 python3-pexpect python3-pip python3-subunit socat texinfo unzip wget xz-utils zstd
> > +sudo apt-get install build-essential chrpath cpio debianutils diffstat file gawk gcc git iputils-ping libacl1 lz4 locales python3 python3-git python3-jinja2 python3-pexpect python3-pip python3-subunit socat texinfo unzip wget xz-utils zstd
> >
>
> Do we actually need lz4 on the system to run a build? This seems a
> little strange to me...

  Good question ... HOSTTOOLS definitely insists that it be there.

rday
Quentin Schulz June 2, 2025, 1:25 p.m. UTC | #3
Hi Robert,

On 6/2/25 3:02 PM, Robert P. J. Day via lists.yoctoproject.org wrote:
> 
> Update the "Ubuntu Essentials package list" by replacing the obsolete
> liblz4-tool package name with the current lz4 package name.
> 
> According to Quentin Schulz:
> 
> "According to pkgs.org, 20.04 lz4 replaces/provides liblz4-tool, same
> for 22.04 and 24.04 and for some reason, liblz4-tool requires lz4
> package, so I think it should be fine to replace it yes.
> 
> "Debian seems to have the same packaging in Bookworm and Bullseye, so
> probably fine to do the change. They only seem to be adding a
> changelog and copyright, and require lz4 package in any case."
> 
> Signed-off-by: Robert P. J. Day <rpjday@crashcourse.ca>
> 

Reviewed-by: Quentin Schulz <quentin.schulz@cherry.de>

Thanks!
Quentin
Richard Purdie June 2, 2025, 1:40 p.m. UTC | #4
On Mon, 2025-06-02 at 09:17 -0400, Robert P. J. Day wrote:
> On Mon, 2 Jun 2025, Richard Purdie via lists.yoctoproject.org wrote:
> 
> > On Mon, 2025-06-02 at 09:02 -0400, Robert P. J. Day via
> > lists.yoctoproject.org wrote:
> > > 
> > > Update the "Ubuntu Essentials package list" by replacing the
> > > obsolete
> > > liblz4-tool package name with the current lz4 package name.
> > > 
> > > According to Quentin Schulz:
> > > 
> > > "According to pkgs.org, 20.04 lz4 replaces/provides liblz4-tool,
> > > same
> > > for 22.04 and 24.04 and for some reason, liblz4-tool requires lz4
> > > package, so I think it should be fine to replace it yes.
> > > 
> > > "Debian seems to have the same packaging in Bookworm and
> > > Bullseye, so
> > > probably fine to do the change. They only seem to be adding a
> > > changelog and copyright, and require lz4 package in any case."
> > > 
> > > Signed-off-by: Robert P. J. Day <rpjday@crashcourse.ca>
> > > 
> > > ---
> > > 
> > > diff --git
> > > a/documentation/tools/host_packages_scripts/ubuntu_essential.sh
> > > b/documentation/tools/host_packages_scripts/ubuntu_essential.sh
> > > index f3388c88c..c4231188b 100644
> > > ---
> > > a/documentation/tools/host_packages_scripts/ubuntu_essential.sh
> > > +++
> > > b/documentation/tools/host_packages_scripts/ubuntu_essential.sh
> > > @@ -1 +1 @@
> > > -sudo apt-get install build-essential chrpath cpio debianutils
> > > diffstat file gawk gcc git iputils-ping libacl1 liblz4-tool
> > > locales python3 python3-git python3-jinja2 python3-pexpect
> > > python3-pip python3-subunit socat texinfo unzip wget xz-utils
> > > zstd
> > > +sudo apt-get install build-essential chrpath cpio debianutils
> > > diffstat file gawk gcc git iputils-ping libacl1 lz4 locales
> > > python3 python3-git python3-jinja2 python3-pexpect python3-pip
> > > python3-subunit socat texinfo unzip wget xz-utils zstd
> > > 
> > 
> > Do we actually need lz4 on the system to run a build? This seems a
> > little strange to me...
> 
>   Good question ... HOSTTOOLS definitely insists that it be there.

I suspect we could/should probably change that (and ASSUME_PROVIDED)
but I your fix is correct as things stand.

Cheers,

Richard
diff mbox series

Patch

diff --git a/documentation/tools/host_packages_scripts/ubuntu_essential.sh b/documentation/tools/host_packages_scripts/ubuntu_essential.sh
index f3388c88c..c4231188b 100644
--- a/documentation/tools/host_packages_scripts/ubuntu_essential.sh
+++ b/documentation/tools/host_packages_scripts/ubuntu_essential.sh
@@ -1 +1 @@ 
-sudo apt-get install build-essential chrpath cpio debianutils diffstat file gawk gcc git iputils-ping libacl1 liblz4-tool locales python3 python3-git python3-jinja2 python3-pexpect python3-pip python3-subunit socat texinfo unzip wget xz-utils zstd
+sudo apt-get install build-essential chrpath cpio debianutils diffstat file gawk gcc git iputils-ping libacl1 lz4 locales python3 python3-git python3-jinja2 python3-pexpect python3-pip python3-subunit socat texinfo unzip wget xz-utils zstd