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

v2.1.0 #196

Merged
merged 59 commits into from
Dec 9, 2024
Merged

v2.1.0 #196

merged 59 commits into from
Dec 9, 2024

Conversation

PLangowski
Copy link
Collaborator

No description provided.

Daniel Mrzygłód and others added 30 commits November 14, 2024 10:10
WARNING: systemd-1_255.4-r0 do_install: Using /home/root as root
	user's home directory is not fully supported by systemd

Set ROOT_HOME to "/root" to avoid such warning.

Signed-off-by: Daniel Mrzygłód <[email protected]>
This patch add to recipe INHIBIT_PACKAGE_DEBUG_SPLIT="1"
this change force system not to build debug packages and do strip
of executables to remove TMPDIR from binary file.

Signed-off-by: Daniel Mrzygłód <[email protected]>
This patch add to recipe INHIBIT_PACKAGE_DEBUG_SPLIT="1"
this change force system not to build debug packages and do strip
of executables to remove TMPDIR from binary file.

Signed-off-by: Daniel Mrzygłód <[email protected]>
golang is adding path if GO_PATH is created and removing this
and adding -trimpath to build process should remove path inside binary

Signed-off-by: Daniel Mrzygłód <[email protected]>
remove CONFIG_INTEL_MENLOW & CONFIG_EFI_VARS are deprecated
add CONFIG_EFIVAR_FS=y

This patch enables new interface for depraceted sysfs one is compatible
with previous one.

Signed-off-by: Daniel Mrzygłód <[email protected]>
This patch fixes:
WARNING: wolfssl-5.7.0-r0 do_package_qa: QA Issue:
	File /usr/lib/libwolfssl.so.42.1.0 in package wolfssl
	contains reference to TMPDIR [buildpaths]

Signed-off-by: Daniel Mrzygłód <[email protected]>
WARNING: gcab-1.6-r0 do_package_qa: QA Issue: File /usr/src/debug/gcab/1.6/libgcab/gcab-enums.c in package gcab-src contains reference to TMPDIR [buildpaths]

Disable QA checkout for this package:
do_package_qa[noexec] = "1"

Signed-off-by: Daniel Mrzygłód <[email protected]>
Signed-off-by: Pawel Langowski <[email protected]>
Signed-off-by: Pawel Langowski <[email protected]>
Signed-off-by: Pawel Langowski <[email protected]>
@m-iwanicki
Copy link
Contributor

m-iwanicki commented Dec 4, 2024

There is something wrong with workflow or runner:

image

If you close/reopen PR or add new commit then either one of those jobs would start (and next one would probably wait forever again). At least that's what happened when I tried it.
As there is no separation in workflow workdir cleanup has to run before build or next job (test) would fail.

Workflows:

Edit: Problem was fixed here: actions/runner#3609 (comment)

Fixes mistake made in f5b42d1

Signed-off-by: Michał Iwanicki <[email protected]>
Signed-off-by: Michał Iwanicki <[email protected]>
@PLangowski
Copy link
Collaborator Author

@m-iwanicki Can you check if everything is fine here? (aside from CI)

@m-iwanicki
Copy link
Contributor

I did 2 weeks ago: #196 (review) and #196 (comment). Except changelog everything else looks ok.

@m-iwanicki
Copy link
Contributor

And check if this is correct srcrev and all changes are in changelog: 629bc4c

@PLangowski
Copy link
Collaborator Author

@m-iwanicki I fixed the changelog. The srcrev looks alright and it seems that the changes are visible in the Changelog.

CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
@m-iwanicki
Copy link
Contributor

@PLangowski Are we following release process from https://docs.zarhus.com/development-process/standard-release-process/#process-steps? If yes then what about Use the generate-changed-recipes script

@PLangowski
Copy link
Collaborator Author

@m-iwanicki since this will be the first release that publishes its manifest, I think we should use the script in the next release, where we will have 2 "official" manifests to compare.

Copy link
Contributor

@m-iwanicki m-iwanicki left a comment

Choose a reason for hiding this comment

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

LGTM

@PLangowski PLangowski merged commit 9f8383f into main Dec 9, 2024
2 of 4 checks passed
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