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

Fix CentOS 7 Build by Checking if FSOPEN_CLOEXEC exists #1299

Merged
merged 1 commit into from
Sep 8, 2023

Conversation

hswong3i
Copy link
Contributor

@hswong3i hswong3i commented Sep 8, 2023

hswong3i added a commit to alvistack/containers-crun that referenced this pull request Sep 8, 2023
    git clean -xdf
    git submodule sync --recursive
    git submodule update --recursive
    git submodule foreach --recursive git clean -xdf
    tar zcvf ../crun_1.9.orig.tar.gz --exclude=.git .
    debuild -uc -us
    cp crun.spec ../crun_1.9-1.spec
    mv ../crun*1.9*.{gz,xz,spec,dsc} /osc/home\:alvistack/containers-crun-1.9/
    rm -rf ../crun*1.9*.*

See containers#1299

Signed-off-by: Wong Hoi Sing Edison <[email protected]>
@hswong3i
Copy link
Contributor Author

hswong3i commented Sep 8, 2023

Copy link
Collaborator

@flouthoc flouthoc left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@giuseppe giuseppe left a comment

Choose a reason for hiding this comment

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

LGTM

@giuseppe
Copy link
Member

giuseppe commented Sep 8, 2023

could you run make clang-format and amend the result?

@hswong3i
Copy link
Contributor Author

hswong3i commented Sep 8, 2023

could you run make clang-format and amend the result?

DONE

hswong3i added a commit to alvistack/containers-crun that referenced this pull request Sep 8, 2023
    git clean -xdf
    git submodule sync --recursive
    git submodule update --recursive
    git submodule foreach --recursive git clean -xdf
    tar zcvf ../crun_1.9.orig.tar.gz --exclude=.git .
    debuild -uc -us
    cp crun.spec ../crun_1.9-1.spec
    mv ../crun*1.9*.{gz,xz,spec,dsc} /osc/home\:alvistack/containers-crun-1.9/
    rm -rf ../crun*1.9*.*

See containers#1299

Signed-off-by: Wong Hoi Sing Edison <[email protected]>
@giuseppe
Copy link
Member

giuseppe commented Sep 8, 2023

still broken.

If you allow changes from maintainers I can fix it up and push a new version

@hswong3i
Copy link
Contributor Author

hswong3i commented Sep 8, 2023

@giuseppe my PR just copy and paste your progress, so simplify for my OBS packaging pipeline and change history.

Maybe I could close this PR, so as original author for the changes you could handle that freely?

hswong3i added a commit to alvistack/containers-crun that referenced this pull request Sep 8, 2023
    git clean -xdf
    git submodule sync --recursive
    git submodule update --recursive
    git submodule foreach --recursive git clean -xdf
    tar zcvf ../crun_1.9.orig.tar.gz --exclude=.git .
    debuild -uc -us
    cp crun.spec ../crun_1.9-1.spec
    mv ../crun*1.9*.{gz,xz,spec,dsc} /osc/home\:alvistack/containers-crun-1.9/
    rm -rf ../crun*1.9*.*

See containers#1299

Signed-off-by: Wong Hoi Sing Edison <[email protected]>
@hswong3i
Copy link
Contributor Author

hswong3i commented Sep 8, 2023

Ok I try to handle make clang-format within a docker container with fedora:latest, as similar as possible as our test cases. It should now looks much better ;-)

Copy link
Member

@giuseppe giuseppe left a comment

Choose a reason for hiding this comment

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

LGTM

@giuseppe giuseppe merged commit c0a40fa into containers:main Sep 8, 2023
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.

[1.9][CentOS 7] FSOPEN_CLOEXEC undeclared
3 participants