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

provide docker file for github builds with cmake #62

Closed
wants to merge 1 commit into from

Conversation

pellcorp
Copy link
Contributor

No description provided.

@@ -102,7 +102,7 @@ libhv.a:

libspdlog.a:
@mkdir -p $(SPDLOG_DIR)/build
@cmake -B $(SPDLOG_DIR)/build -S $(SPDLOG_DIR)/
@cmake -B $(SPDLOG_DIR)/build -S $(SPDLOG_DIR)/ -DCMAKE_CXX_COMPILER=$(CXX)
Copy link
Owner

Choose a reason for hiding this comment

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

Is this needed? Are line 5 and 6 where CC/CXX are defined no enough?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

In my local environment I could not get cmake to work without it, but perhaps that was only on the command line.

@@ -0,0 +1,18 @@
FROM ubuntu:22.04
Copy link
Owner

Choose a reason for hiding this comment

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

Maybe just do FROM ballaswag/guppydev:latest then RUN DEBIAN_FRONTEND=noninteractive apt-get update && \ apt-get install -y --no-install-recommends cmake apt-get clean all && \ apt-get -y autoremove

Will try this and publish one with cmake

@pellcorp
Copy link
Contributor Author

I am closing this, as it seems like you have it sorted.

@pellcorp pellcorp closed this Apr 17, 2024
@pellcorp pellcorp deleted the jp_dockerfile branch December 21, 2024 13:35
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.

2 participants