NcISO is not computing the ISO fileIdentifier correctly from the id and naming_authority about thredds HOT 6 CLOSED

I believe this issue (and PR 122) should be on Unidata/threddsIso

from thredds.

... Or at least on both?

from thredds.

@ethanrd, Last time we fixed Unidata/threddsIso it didn't make it into the next THREDDS version, so I figured I'd make the change here this time. It seems kind of crazy doing PRs for both, but if that's what it takes I'm willing to do that. Do you want two issues also?

from thredds.

@rsignell-usgs Yes, it is crazy. And it shouldn't be necessary. But until the THREDDS release engineering process is more fully automated (and lots of work has been happening around this lately) there is some level of work to get threddsIso changes into thredds. Which leaves it open for being missed. When the change is only the threddsIso XSLT files, making a PR to each repository decreases the risk of it being missed. Yes, its an ugly kludge, but its what we have at the moment.

Perhaps @cwardgar or @lesserwhirls has more thoughts on the automation of pulling threddsIso (and ncWMS) into the thredds build/release.

from thredds.

We need to have a plugin layer for TDS services which would allow us to
completely remove any ncIso or ncWms specific code from the THREDDS
codebase. This is something I'm eyeballing for THREDDS 5.0.

On Wed, Apr 15, 2015 at 10:25 AM, Ethan Davis [email protected]
wrote:

@rsignell-usgs https://github.com/rsignell-usgs Yes, it is crazy. And
it shouldn't be necessary. But until the THREDDS release engineering
process is more fully automated (and lots of work has been happening around
this lately) there is some level of work to get threddsIso changes into
thredds. Which leaves it open for being missed. When the change is only the
threddsIso XSLT files, making a PR to each repository decreases the risk of
it being missed. Yes, its an ugly kludge, but its what we have at the
moment.

Perhaps @cwardgar https://github.com/cwardgar or @lesserwhirls
https://github.com/lesserwhirls has more thoughts on the automation of
pulling threddsIso (and ncWMS) into the thredds build/release.

—
Reply to this email directly or view it on GitHub
#121 (comment).

from thredds.

Fixed by #122

from thredds.