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

prolog of Spec files need to point to new Jenkins addr #1408

Closed
sydb opened this issue Dec 3, 2015 · 10 comments
Closed

prolog of Spec files need to point to new Jenkins addr #1408

sydb opened this issue Dec 3, 2015 · 10 comments

Comments

@sydb
Copy link
Member

sydb commented Dec 3, 2015

"tei.oucs.ox.ac.uk" -> "jenkins.tei-c.org" passim, right?

@sydb sydb self-assigned this Dec 3, 2015
@jamescummings
Copy link
Member

Not just that...we need to make sure the URL is right as well -- we renamed a jenkins job I believe.

This needs to wait:
a) until TEI Sysadmin has added the DNS aliases for the subdomains
b) until the pure odd branch is merged in (as a big piece of work... we might as well do that first and then this one mass change rather than have to re-do it or complicate the merge).

-James

@jamescummings
Copy link
Member

@martindholmes
Copy link
Contributor

This will have to be done carefully to avoid screwing up the Pure-ODD merge. Last time I did it, I gave Lou a lot of extra work to do.

@lb42
Copy link
Member

lb42 commented Dec 14, 2015

Yes. Let's defer it till after the Pure ODD merge, plz.

@jamescummings
Copy link
Member

Yes, as the second comment on the ticket (mine) says in b) it needs to wait until the pure odd branch is merged.

@martindholmes
Copy link
Contributor

Note that the Makefile for P5 does a sed replace for the old URL and replaces it with the release version; this will fail as soon as the URL changes, so that'll have to be updated at the same time.

@hcayless
Copy link
Member

Pure is in, so changing the status to Go.

@jamescummings
Copy link
Member

@hcayless has the Makefile for P5 also changed (See @martindholmes's comment above)?

@hcayless
Copy link
Member

Yes. See a33843b.

@jamescummings
Copy link
Member

Thanks! Looks right to me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants