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.

Sign up for GitHub

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

Jump to bottom

Remove dead/unused keys from framework.properties #575

Closed
ahonor opened this issue Oct 24, 2013 · 2 comments
Closed

Remove dead/unused keys from framework.properties #575

ahonor opened this issue Oct 24, 2013 · 2 comments
Assignees
@ahonor
Labels
configuration enhancement install
Milestone
2.0.0-beta1

Comments

@ahonor
Copy link
Contributor

ahonor commented Oct 24, 2013

There are many old/crufty properties left in the framework.properties. Remove them:

framework.application.libpath
framework.application.properties
framework.email.tolist
framework.email.from
framework.email.replyto
framework.email.mailhost
framework.email.mailport
framework.email.user =
framework.email.password =
framework.email.ssl = false
framework.email.failonerror = true
framework.crypto.keystore.filename
framework.crypto.keystore.password
framework.crypto.jarSigning.aliasName
framework.node
framework.rdeck.version
framework.rdeck.dir
framework.log.format
framework.src.dir
framework.server.url
@ghost ghost assigned ahonor Oct 24, 2013
@ahonor
Copy link
Contributor Author

ahonor commented Oct 25, 2013

Also consolidate framework.node/server properties. This also ties to rd-setup.

@ahonor
Copy link
Contributor Author

ahonor commented Oct 26, 2013

See 356aeef

@ahonor ahonor closed this as completed Oct 26, 2013
ahonor added a commit that referenced this issue Oct 29, 2013
@ahonor
removed a few more configuration settings from framework.properties.
77de5ce 
Removed some app config related settings from the framework.properties
file as these are largely internal and not really useful to end users.
End users can still set these settings but they are no longer part
of the installer config.

Fixes #575
ahonor added a commit that referenced this issue Oct 30, 2013
@ahonor
removed internal settings from system package config templates.
ee7485b 
See #575
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ahonor ahonor

Labels
configuration enhancement install
Projects
None yet
Milestone
2.0.0-beta1
Development

No branches or pull requests
1 participant
@ahonor