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

Ubuntu Launchpad CI build failling #73

Open
jfmcarreira opened this issue Apr 27, 2024 · 8 comments
Open

Ubuntu Launchpad CI build failling #73

jfmcarreira opened this issue Apr 27, 2024 · 8 comments

Comments

@jfmcarreira
Copy link
Collaborator

Fix the CI service of the ubuntu launchpad

@jfmcarreira
Copy link
Collaborator Author

@alexfikl Do you use ubuntu for development? I am having problems making the build pass using qmake. however the github action you added passes here.

@alexfikl
Copy link
Contributor

No, I'm afraid I'm using arch (and their packaging is quite different from what I recall).

Like you said the CI uses Ubuntu, the 22.04 LTS from what it says on https://github.com/actions/runner-images?tab=readme-ov-file#available-images, so it should work. What errors are you seeing?

Also, what's the "Ubuntu Launchpad CI"? Are the logs from that available?

@jfmcarreira
Copy link
Collaborator Author

the error is pretty easy to spot. :)
https://launchpadlibrarian.net/726492683/buildlog_ubuntu-jammy-amd64.ktikz_0.13.1-202404262122-5696d92~ubuntu22.04.1_BUILDING.txt.gz

In case it requires login:
image

However, I do not get why it is not finding poppler.

I am setting up a ubuntu virtual machine to test the issue. The other day I did a bunch of force push, since I was playing around to see if i could fix it. I also changed some minor things to make it pass.

I am targetting for at least Ubuntu 22.04.

Maybe it could be interesting if we have a package for arch :) not sure how hard it is.

@alexfikl
Copy link
Contributor

alexfikl commented Apr 27, 2024

the error is pretty easy to spot. :) launchpadlibrarian.net/726492683/buildlog_ubuntu-jammy-amd64.ktikz_0.13.1-202404262122-5696d92~ubuntu22.04.1_BUILDING.txt.gz

Yep, that's very clear! The Github CI isn't doing anything special from what I can tell, so not sure why it doesn't find poppler includes on the Launchpad CI. Especially since they seem to be using the same ubuntu version..

I am setting up a ubuntu virtual machine to test the issue. The other day I did a bunch of force push, since I was playing around to see if i could fix it. I also changed some minor things to make it pass.

I can try the docker image locally to see if there's any difference there :\

Maybe it could be interesting if we have a package for arch :) not sure how hard it is.

There's already an official package for Arch: https://archlinux.org/packages/extra/x86_64/ktikz/ 😁

@alexfikl
Copy link
Contributor

alexfikl commented Apr 27, 2024

(mostly note to self) Just from looking at the logs, the Github CI clearly has a bunch of (repeated) poppler includes that the Launchpad run is missing:

# Github
g++ -c -pipe -I/usr/include/poppler/qt5 -I/usr/include/poppler -I/usr/include/poppler/qt5 -I/usr/include/poppler -I/usr/include/poppler/qt5 -I/usr/include/poppler -O2 -D_REENTRANT -Wall -Wextra [...]
# Launchpad
g++ -c -pipe -O2 -D_REENTRANT -Wall -Wextra [...]

The includes should be found in qmake/findpoppler.pri using pkg-config:

    POPPLERINCLUDES = $$system($$PKG_CONFIG --cflags poppler-qt5)

EDIT: The Github images have a lot of packages installed and it's not clear how they compare to the Launchpad CI
https://github.com/actions/runner-images/blob/main/images/ubuntu/Ubuntu2204-Readme.md

@jfmcarreira
Copy link
Collaborator Author

Yeah i noticed that too. seems like in the ubuntu launchpad the pkg-config for the poppler is not giving the right result

@jfmcarreira
Copy link
Collaborator Author

I think it is a missing dependency on pkg-config

@jfmcarreira
Copy link
Collaborator Author

Solved the build issue. Now just adding back a dependency i removed and didnt allow to process icons :)

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

No branches or pull requests

2 participants