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

Build in Github Actions not successful with Ubuntu-20.04.05 #176

Closed
JuJup opened this issue Sep 9, 2022 · 2 comments · Fixed by #177
Closed

Build in Github Actions not successful with Ubuntu-20.04.05 #176

JuJup opened this issue Sep 9, 2022 · 2 comments · Fixed by #177

Comments

@JuJup
Copy link

JuJup commented Sep 9, 2022

Hey there!
I started to tinker around with CustomPiOS and Github Actions and couldn't make it work.
After checking other repositories using CustomPiOS I could see, that the builds fail since September 7 (e.g. OctoPi).
The only difference I can see is the change of the ubuntu version in the runner. With 20.04.04 it seemed to work, now the runner is using 20.04.05. Unfortunately, I can't tell the runner which exact version it should use.

Furthermore, the error happens somewhere in the /patches/fix-163-autologin-console.patch. I could see projects, where this patch is not called and their build is running successfully.
Unfortunately, I just started digging into this project, so I have no ideas in mind how to fix this.
Maybe somebody can help :)

A part of the log before the error occurs:

Setup default user and password ...
+ echo -e -n '\e[0m'
++ get_os_version
++ local os_version
++ grep -c buster /etc/os-release
+ '[' 0 == 0 ']'
+ create_userconf
+ local pw_encrypt
+ '[' -n pi ']'
++ echo raspberry
++ openssl passwd -6 -stdin
+ pw_encrypt='$6$rmdZ8JK34QLLzWu1$vcar46INDuAj/0J2q./iQvCdbJaWxxMcBNc/kg72P3ICINRxN16HeUV9nAkqgHptxJvOPRac.m/fGNtgw696X0'
+ echo 'pi:$6$rmdZ8JK34QLLzWu1$vcar46INDuAj/0J2q./iQvCdbJaWxxMcBNc/kg72P3ICINRxN16HeUV9nAkqgHptxJvOPRac.m/fGNtgw696X0'
+ apt install --yes --only-upgrade userconf-pi

WARNING: apt does not have a stable CLI interface. Use with caution in scripts.

Reading package lists...
Building dependency tree...
Reading state information...
userconf-pi is already the newest version (0.5).
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.
+ unpack /filesystem/patches /patches root
+ from=/filesystem/patches
+ to=/patches
+ owner=
+ '[' 3 -gt 2 ']'
+ owner=root
+ mkdir -p /tmp/unpack/
+ cp -v -r --preserve=mode,timestamps /filesystem/patches/. /tmp/unpack/
'/filesystem/patches/./fix-163-autologin-console.patch' -> '/tmp/unpack/./fix-163-autologin-console.patch'
+ '[' -n root ']'
+ chown -hR root:root /tmp/unpack/
+ cp -v -r --preserve=mode,ownership,timestamps /tmp/unpack/. /patches
'/tmp/unpack/.' -> '/patches'
'/tmp/unpack/./fix-163-autologin-console.patch' -> '/patches/fix-163-autologin-console.patch'
+ rm -r /tmp/unpack
+ pushd /usr/bin
/usr/bin /
+ patch -p1
patching file cancel-rename
Hunk #1 FAILED at 25.
1 out of 1 hunk FAILED -- saving rejects to file cancel-rename.rej
++++ echo_red 'build failed, unmounting image...'
++++ echo -e -n '\e[91m'
++++ echo build failed, unmounting image...

Regards

KwadFan added a commit to mainsail-crew/MainsailOS that referenced this issue Sep 9, 2022
Proof of Concept Test for
guysoft/CustomPiOS#176

Signed-off-by: Stephan Wendel <me@stephanwe.de>
@guysoft
Copy link
Owner

guysoft commented Sep 9, 2022

Hey, related:
guysoft/FullPageOS#440
The new Rpi image breaks the current build system because they changed the userless repair script.

See: #163

KwadFan added a commit to KwadFan/CustomPiOS that referenced this issue Sep 9, 2022
This fixes guysoft#176

Location, linewise has changed in cancel-rename script.

Uses now a generic method to patch

Signed-off-by: Stephan Wendel <me@stephanwe.de>
@KwadFan
Copy link
Contributor

KwadFan commented Sep 9, 2022

Hey @guysoft ,
I allowed my self to do some of your fix work ;)

Happy Weekend and regards.

guysoft added a commit that referenced this issue Sep 9, 2022
fix: fixes build error in base module
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants