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

Feature/add client build artifacts and run packr #55

Merged
merged 17 commits into from
Feb 15, 2019

Conversation

hplatou
Copy link
Contributor

@hplatou hplatou commented Feb 8, 2019

Jenkins will now build archives containing cse-server-go executable and dependent libraries both for Windows and Linux. 'rpath' modified on Linux libraries to allow 'automatic' library loading.

@hplatou hplatou self-assigned this Feb 11, 2019
@hplatou
Copy link
Contributor Author

hplatou commented Feb 11, 2019

This pr should not be merged until feature/151-build-conan-linux-binaries is merged in cse-core. Also, after that the channel of the dependency in the conanfile.txt should be changed to "master"

Copy link
Member

@ljamt ljamt left a comment

Choose a reason for hiding this comment

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

Awesome!

When downloading and running the windows artifact from jenkins I get errors about missing boost_chrono-vc141-mt-x64-1_66.dll and boost_date_time-vc141-mt-x64-1_66.dll

Copy link
Member

@eidekrist eidekrist left a comment

Choose a reason for hiding this comment

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

LGTM! When testing locally I also get the same errors as Levi, missing:

  • boost_date_time-vc141-mt-x64-1_66.dll
  • boost_chrono-vc141-mt-x64-1_66.dll

@hplatou hplatou merged commit a0884be into master Feb 15, 2019
@hplatou hplatou deleted the feature/add-client-build-artifacts-and-run-packr branch February 15, 2019 17:23
hplatou pushed a commit that referenced this pull request May 13, 2020
…buttons-for-plot-adjustments

feature #43: split plot creation and plot variable addition
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