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

Remove reference to old version of netcdf library - talend can pick i… #505

Merged
merged 1 commit into from
Mar 20, 2017

Conversation

jonescc
Copy link
Contributor

@jonescc jonescc commented Mar 14, 2017

…t up from installed components

Most likely cause for #504 and #501.

@jonescc
Copy link
Contributor Author

jonescc commented Mar 15, 2017

@bpasquer - can you review this? If we build anfog_rt again without this fix, we'll end up with issues like #504 and #501 again.

@bpasquer
Copy link
Contributor

Hold on Craig, are you fixing ANFOG_DM or ANFOG_RT or both?

@jonescc
Copy link
Contributor Author

jonescc commented Mar 15, 2017

ANFOG_RT includes an old version of the netcdf java library. This was recently built in jenkins causing the old version of the netcdf java library to be copied to the harvester build library. ANFOG_DM and AODN_RAN_CTD were then built in jenkins and the old version of the library was copied from the harvester build library along with the required version. This then caused #504 and #501. So not fixing ANFOG_DM but stopping ANFOG_RT from causing similar issues in future.

@bpasquer
Copy link
Contributor

ok, i see

@bpasquer bpasquer merged commit 2cbe76f into master Mar 20, 2017
@bpasquer
Copy link
Contributor

It solved the issue on the PO box. Could you please deploy on RC @lbesnard

@lbesnard
Copy link
Contributor

will be deployed on RC at lunch

@bpasquer bpasquer deleted the fix-anfog_rt_netcdf_references branch July 25, 2017 03:45
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.

4 participants