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 Conan Build - Consistent Repo Hash IDs #52

Merged
merged 2 commits into from
Dec 20, 2023
Merged

Conversation

callendorph
Copy link
Collaborator

This refactors the conanfile.py to get a consistent hash id on different platforms.

Note also - that git config core.autocrlf false might also be necessary on windows so that the conanfile.py does not have CRLF line endings.

I've added a script to help run the build on both linux/mac/windows.
I've confirmed that I get the same hash on mac and windows. I'll be checking linux shortly.

The problem was that we were pushing the built binary through
the `export_sources` step. This was getting hashed and causing
the repo hashs for windows, linux and mac to all be different.
This caused problems in distribution through artifactory because
I would never get the appropriate binary.
@callendorph callendorph requested a review from jwatson0 December 19, 2023 19:43
@callendorph
Copy link
Collaborator Author

Confirmed build on Linux is the same repo id.

Copy link
Collaborator

@jwatson0 jwatson0 left a comment

Choose a reason for hiding this comment

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

I think this is good for now, but passing in the SLM_ROOT_DIR seems hacky, as you said.

The next step would be enabling the conan build to start the compile. Getting that to work may clear up some of the hacky issues.

@callendorph
Copy link
Collaborator Author

The next step would be enabling the conan build to start the compile. Getting that to work may clear up some of the hacky issues.

Agreed - that is where I want to get to. It will have to be for the next release though.

@callendorph callendorph merged commit 2ff857f into master Dec 20, 2023
1 check passed
@callendorph callendorph deleted the cja/fix_conan_build branch December 20, 2023 00:23
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