Skip to content

Conversation

@mkopec
Copy link
Member

@mkopec mkopec commented Mar 30, 2022

No description provided.

@mkopec mkopec requested a review from macpijan March 30, 2022 11:01
@mkopec mkopec self-assigned this Mar 30, 2022
@mkopec mkopec force-pushed the dasharo-ci branch 20 times, most recently from afa1d75 to 06e5b69 Compare March 30, 2022 15:07
@mkopec mkopec marked this pull request as draft March 31, 2022 08:08
@macpijan
Copy link

macpijan commented Apr 4, 2022

@mkopec is it still a draft?

@mkopec
Copy link
Member Author

mkopec commented Apr 4, 2022

@macpijan Yes, at the moment it builds for debian only. Won't work for ubuntu

@mkopec mkopec force-pushed the dasharo-ci branch 5 times, most recently from 9136883 to 7b30f38 Compare April 4, 2022 13:27
@mkopec mkopec force-pushed the dasharo-ci branch 5 times, most recently from 6554fa6 to b2e04ea Compare April 4, 2022 14:05
@mkopec mkopec marked this pull request as ready for review April 4, 2022 14:27
mkopec added 8 commits April 4, 2022 16:28
For building packages only.

Signed-off-by: Michał Kopeć <michal.kopec@3mdeb.com>
Signed-off-by: Michał Kopeć <michal.kopec@3mdeb.com>
Signed-off-by: Michał Kopeć <michal.kopec@3mdeb.com>
Signed-off-by: Michał Kopeć <michal.kopec@3mdeb.com>
Signed-off-by: Michał Kopeć <michal.kopec@3mdeb.com>
Signed-off-by: Michał Kopeć <michal.kopec@3mdeb.com>
Signed-off-by: Michał Kopeć <michal.kopec@3mdeb.com>
Signed-off-by: Michał Kopeć <michal.kopec@3mdeb.com>
@mkopec
Copy link
Member Author

mkopec commented Apr 4, 2022

@macpijan Rebased, ready for review. Packages are built for Ubuntu 20.04 and can be installed correctly. Only the -dev packages don't install correctly due to a missing dependency, but these aren't really necessary for regular usage.

</distro>
<distro id="ubuntu">
<package variant="x86_64" />
<package variant="20.04" />
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mkopec Why do we need this change and it was not presetn in the usptream? Is it upstreamable?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fwupd upstream doesn't build packages for ubuntu. I tried adding a new variant ubuntu-20.04 with a different CI script for ubuntu, but at the moment this code is unused.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, maybe this 20.04 does not make much sense. But we need to provide OS=ubuntu-x86_64 as a variable, so the contrib is generated correctly.

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 this pull request may close these issues.

4 participants