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

minor installer improvements #5294

Closed
pameyer opened this issue Nov 8, 2018 · 6 comments
Closed

minor installer improvements #5294

pameyer opened this issue Nov 8, 2018 · 6 comments
Assignees

Comments

@pameyer
Copy link
Contributor

pameyer commented Nov 8, 2018

Noticed 2 minor places to improve installer when doing a manual install:

  • on some systems, restart-domain is less robust than stop-domain; start-domain
  • if the domain isn't running, the installer attempts to sudo to configured glassfish user (which may not be in sudoers) without checking if it's running as the glassfish user
@kcondon
Copy link
Contributor

kcondon commented Nov 8, 2018

@pameyer Nice avatar

@pameyer
Copy link
Contributor Author

pameyer commented Nov 8, 2018

@kcondon thanks - I was getting confused with having >4 automaticly generated ones in different systems and decided to standardize.

@kcondon
Copy link
Contributor

kcondon commented Nov 9, 2018

@pameyer Built and ran the installer. Encountered a bunch of errors when it tried to run:

./install
"my" variable $mail_server_status masks earlier declaration in same scope at ./install line 1154.
syntax error at ./install line 966, near "else"
syntax error at ./install line 970, near "}"
Global symbol "$success" requires explicit package name at ./install line 984.
Global symbol "$success" requires explicit package name at ./install line 988.
Global symbol "$success" requires explicit package name at ./install line 1071.
syntax error at ./install line 1072, near "}"
syntax error at ./install line 1119, near "}"
Can't use global @_ in "my" at ./install line 1122, near "shift @_"
syntax error at ./install line 1149, near "}"
syntax error at ./install line 1184, near "}"
./install has too many errors.

@pameyer
Copy link
Contributor Author

pameyer commented Nov 9, 2018

@kcondon Sorry about that; I messed up my back port. Should be fixed, I'm rechecking this branch to confirm.

@pameyer pameyer self-assigned this Nov 9, 2018
@pameyer pameyer removed their assignment Nov 9, 2018
@kcondon
Copy link
Contributor

kcondon commented Nov 9, 2018

@pameyer OK, that works now, thanks!
Btw, I noticed some weirdness around prompts and the final DataCite message, indicating you needed to contact DataCite to get an account, even though it prompted me for the account earlier on, which I provided values for.

The weird prompts look to be kind of duplicate string combining around datacite questions:
Datacite usernameDataCite or EZID username. Only necessary for publishing / minting DOIs.[dataciteuser]
Datacite passwordDataCite or EZID account password.[datacitepassword]
Datacite URLDataCite or EZID URL. Probably https://mds.datacite.org[https://mds.test.datacite.org]

Any thoughts on those or should they be a separate ticket?

@pameyer
Copy link
Contributor Author

pameyer commented Nov 9, 2018

@kcondon I didn't notice the prompts (although I did see the message about them needing to be configured at the end). I'd lean towards those being a separate issue.

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

No branches or pull requests

4 participants