-
Notifications
You must be signed in to change notification settings - Fork 284
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
drop -noroot for recent FLUENT versions, honor installopts, enable -debug #582
Conversation
Refer to this link for build results (access rights to CI server needed): |
if LooseVersion(self.version) < LooseVersion('15.0'): | ||
extra_args += '-noroot' | ||
|
||
cmd = "./INSTALL %s -debug -silent -install_dir %s %s" % (extra_args, self.installdir, self.cfg['installopts']) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we want debug enabled always?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
doesn't hurt, I guess
the real solution for large (debug) logs is supporting automatically zipping them at the end of the installation ;)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
debug refers too the install process or the actual application being installed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the install process
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
forget the remark than 😉
retested with all existing FLUENT easyconfigs, nothing was broken, so going in Thanks for the review @wpoely86! |
drop -noroot for recent FLUENT versions, honor installopts, enable -debug
@wpoely86: please review, well (re)test all existing FLUENT easyconfigs with this in place