-
Notifications
You must be signed in to change notification settings - Fork 34
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
SUSEconfig for postfix and sendmail removed #59
Changes from all commits
0bb4ee5
89152f0
7b1a04b
c50ed52
0ba8320
48ef76d
16eaa92
4fdcbeb
143f558
11f0658
fb62d64
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1 +1 @@ | ||
2.24.14 | ||
2.24.15 |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -45,28 +45,12 @@ textdomain "network"; | |
* @return true if success | ||
*/ | ||
define boolean RunSuSEconfig() { | ||
y2milestone("Running SuSEconfig ..."); | ||
SCR::Execute(.target.bash, "/sbin/SuSEconfig --module apache 2>/dev/null"); | ||
SCR::Execute(.target.bash, "/sbin/SuSEconfig --module sendmail 2>/dev/null"); | ||
SCR::Execute(.target.bash, "/sbin/SuSEconfig --module postfix 2>/dev/null"); | ||
y2milestone("Updating sendmail and/or postfix configuration."); | ||
SCR::Execute(.target.bash, "/usr/lib/sendmail.d/update 2>/dev/null"); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Maybe link to documentation in comment? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Would be nice. But I don't know any documentation ;-) |
||
SCR::Execute(.target.bash, "/usr/sbin/config.postfix 2>/dev/null"); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. As above. Just an idea. |
||
return true; | ||
} | ||
|
||
/** | ||
* Run one SuSEconfig module | ||
* @param modul SuSEconfig module | ||
* @return true if success | ||
*/ | ||
define boolean RunSuSEconfigModule(string modul) { | ||
if(modul == nil || modul == "") { | ||
y2error("Wrong SuSEconfig module: %1", modul); | ||
return false; | ||
} | ||
y2milestone("Running SuSEconfig module %1", modul); | ||
return (SCR::Execute(.target.bash, sformat("/sbin/SuSEconfig --module %1", modul)) == 0); | ||
} | ||
|
||
|
||
/** | ||
* Link detection | ||
* @return true if link found | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,17 +1,10 @@ | ||
Dump RunSuSEconfig | ||
Execute .target.bash "/sbin/SuSEconfig --module apache 2>/dev/null" 0 | ||
Execute .target.bash "/sbin/SuSEconfig --module sendmail 2>/dev/null" 0 | ||
Execute .target.bash "/sbin/SuSEconfig --module postfix 2>/dev/null" 0 | ||
Execute .target.bash "/usr/lib/sendmail.d/update 2>/dev/null" 0 | ||
Execute .target.bash "/usr/sbin/config.postfix 2>/dev/null" 0 | ||
Return true | ||
Execute .target.bash "/sbin/SuSEconfig --module apache 2>/dev/null" 1 | ||
Execute .target.bash "/sbin/SuSEconfig --module sendmail 2>/dev/null" 1 | ||
Execute .target.bash "/sbin/SuSEconfig --module postfix 2>/dev/null" 1 | ||
Execute .target.bash "/usr/lib/sendmail.d/update 2>/dev/null" 1 | ||
Execute .target.bash "/usr/sbin/config.postfix 2>/dev/null" 1 | ||
Return true | ||
Dump RunSuSEconfigModule | ||
Execute .target.bash "/sbin/SuSEconfig --module postfix" 0 | ||
Return true | ||
Execute .target.bash "/sbin/SuSEconfig --module postfix" 1 | ||
Return false | ||
Dump SetupSMPPPD | ||
Return true | ||
Return false |
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.
Minor: without "..."