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

Fedora build #441

Merged
merged 18 commits into from
Nov 24, 2021
Merged

Fedora build #441

merged 18 commits into from
Nov 24, 2021

Conversation

eidins
Copy link
Contributor

@eidins eidins commented Nov 4, 2021

Replacement for #410 with rebase and conflict resolved

@eidins
Copy link
Contributor Author

eidins commented Nov 4, 2021

cmake ..
make
make package

All work and the RPM installs fine.

CMakeLists.txt Outdated Show resolved Hide resolved
.github/workflows/build.yml Outdated Show resolved Hide resolved
@metsma
Copy link
Contributor

metsma commented Nov 16, 2021

If you plan build digidoc4-client you probably need also save the artifacts. If not we can merge

@metsma
Copy link
Contributor

metsma commented Nov 18, 2021

There are some issues with the artefacts
"Build on Fedora to fedora:33
No files were found with the provided path: libdigidocpp*.rpm. No artifacts will be uploaded."

@metsma metsma self-requested a review November 18, 2021 18:12
uses: actions/upload-artifact@v2
with:
name: rpms
path: libdigidocpp*.rpm
Copy link
Contributor

Choose a reason for hiding this comment

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

Build on Fedora to fedora:33
No files were found with the provided path: libdigidocpp*.rpm. No artifacts will be uploaded.

Copy link
Contributor

Choose a reason for hiding this comment

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

path: build/libdigidocpp*.rpm ??

@metsma
Copy link
Contributor

metsma commented Nov 19, 2021

Lets see if this also adds config files to correct place
set(CPACK_RPM_SOURCE_PKG_BUILD_PARAMS -DCMAKE_INSTALL_SYSCONFDIR=/etc)

.github/workflows/build.yml Outdated Show resolved Hide resolved
CMakeLists.txt Outdated Show resolved Hide resolved
eidins and others added 2 commits November 21, 2021 13:15
Co-authored-by: Raul Metsma <raul@metsma.ee>
Co-authored-by: Raul Metsma <raul@metsma.ee>
@kristelmerilain kristelmerilain merged commit 076f6e5 into open-eid:master Nov 24, 2021
@eidins eidins deleted the patch-1 branch January 21, 2022 19:13
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.

3 participants