Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: v0.14.0 Unable to build .deb package #458

Closed
gardouille opened this issue Oct 2, 2023 · 4 comments · Fixed by #460
Closed

ci: v0.14.0 Unable to build .deb package #458

gardouille opened this issue Oct 2, 2023 · 4 comments · Fixed by #460

Comments

@gardouille
Copy link
Contributor

If eza fails to compile, or if there is a problem during the build process, then please include the following information in your report:

  • The exact eza commit you are building : 9a9df55
  • Your operating system and hardware platform : Docker debian/bookworm

If you are seeing compilation errors, please include the output of the build process.

./devtools/deb-package.sh output :

checkout tag v0.14.0
build man pages
sh: 1: Syntax error: "(" unexpected
error: Recipe `man` failed on line 80 with exit code 2

To reproduce :

docker run -it "debian:bookworm" bash

apt-get update -yq && apt-get install -yq aptitude curl git wget gzip lintian pandoc gpg lsb-release

# Just
wget -qO - 'https://proget.makedeb.org/debian-feeds/prebuilt-mpr.pub' | gpg --dearmor | tee /usr/share/keyrings/prebuilt-mpr-archive-keyring.gpg 1> /dev/null
echo "deb [arch=all,$(dpkg --print-architecture) signed-by=/usr/share/keyrings/prebuilt-mpr-archive-keyring.gpg] https://proget.makedeb.org prebuilt-mpr $(lsb_release -cs)" | tee /etc/apt/sources.list.d/prebuilt-mpr.list
apt-get update -yq && aptitude --quiet --assume-yes install just

git clone https://github.com/eza-community/eza && cd eza

./devtools/deb-package.sh
@gardouille
Copy link
Contributor Author

And to be more precise, the error comes from just command to build manpages :

cd eza
just man

Produce :

sh: 1: Syntax error: "(" unexpected
error: Recipe `man` failed on line 80 with exit code 2

Everything worked fine before the commit c496bae

@gierens
Copy link
Member

gierens commented Oct 2, 2023

Hm, my initial guess would be that the bash version in the container is too old maybe ... can you tell us what version that is? I'm running bash 5.1.16 on my machine and there it works.

Also, as a side note ... the deb-package.sh is actually just intended for us to build the packages for the deb repo not for users. But this is still a good point in case we want to CI this, too.

@gardouille
Copy link
Contributor Author

bash --version :

GNU bash, version 5.2.15(1)-release (x86_64-pc-linux-gnu)
Copyright (C) 2022 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>

This is free software; you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.

And just --version :

just --version
just 1.14.0

And yeah, i noticed deb.gierens.de (thanks for that 👍 ) but has i don't have access to this domain at work… I managed to have my own auto-build of Eza .deb package based on a tiny script + Docker and the tools available at work ^^

I will comment in the PR for the rest :)

@gierens
Copy link
Member

gierens commented Oct 2, 2023

just updated the deb repo by the way ... now ships v0.14.0 ;)

gardouille added a commit to gardouille/eza that referenced this issue Oct 2, 2023
This also avoid "useless cat" and allow `just` command to build
manpages.

Resolves eza-community#458
gierens pushed a commit to gardouille/eza that referenced this issue Oct 9, 2023
This also avoid "useless cat" and allow `just` command to build
manpages.

Resolves eza-community#458
gierens added a commit to gardouille/eza that referenced this issue Oct 9, 2023
This also avoid "useless cat" and allow `just` command to build
manpages.

Resolves eza-community#458

Co-authored-by: gierens <sandro@gierens.de>
Co-authored-by: Gardouille <gardouille@gmail.com>
MartinFillon pushed a commit to MartinFillon/eza that referenced this issue Oct 11, 2023
This also avoid "useless cat" and allow `just` command to build
manpages.

Resolves eza-community#458

Co-authored-by: gierens <sandro@gierens.de>
Co-authored-by: Gardouille <gardouille@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants