-
Notifications
You must be signed in to change notification settings - Fork 5
Check other platforms #128
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
base: master
Are you sure you want to change the base?
Check other platforms #128
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice to see that you got it to work!
- name: install dependencies | ||
run: | | ||
brew install cmake boost doxygen | ||
git clone https://github.com/tjhei/numdiff.git |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's the repository of my friend Timo Heister, but it's not the master repository for the numdiff project. Is that what you had in mind?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh, that's weird - it's the first thing that comes up when I search for numdiff. Can you point me to the master for the numdiff project?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The project lives here: https://www.nongnu.org/numdiff/
You should be able to get it via wget
from http://download-mirror.savannah.gnu.org/releases/numdiff/ as a .tar.gz file that you can unpack via tar xvzf file.tar.gz
.
CMakeLists.txt
Outdated
# Set paths correctly to take BOOST from wherever it is | ||
# installed on the current system | ||
FIND_PACKAGE(Boost 1.39 REQUIRED) | ||
FIND_PACKAGE(Boost REQUIRED COMPONENTS system) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Was there a particular reason to remove the minimum version as a requirement?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Still trying to figure it out, windows cmake fails saying it could not find boost. I found a stack overflow post suggesting this might help, but it's still failing...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's try to figure that out on Wed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Try undoing this change.
Co-authored-by: Wolfgang Bangerth <[email protected]>
If you look at the current version, |
Yep, I think you're right. Let me try something else. |
@bangerth Any thoughts at this point? The environment variables appear to be set up correctly, but the error message persists. I have looked around for solutions but I haven't found anything conclusive. Here is a windows setup guide from the boost docs. Some SO conversations seems suggest that an additional install is required, e.g. |
I'm colossally ignorant about Windows, so really don't know what to make of the error. That said, cmake puts output from its failing checks into
where the I don't know whether there is a corresponding way on the Windows shell. The alternative is to use the "artifacts" system of CI where certain output files are uploaded I don't know off-hand how this is set up, but I'm sure you can find it online -- the file |
Upload Mac output files as artifacts to see what's going wrong. |
No description provided.