Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 194546 - www-servers/apache-2.2.6-r1 initscript missing ${APACHE2_OPTS} in places
Summary: www-servers/apache-2.2.6-r1 initscript missing ${APACHE2_OPTS} in places
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High normal
Assignee: Apache Team - Bugzilla Reports
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-10-02 20:07 UTC by Mark J. Olah
Modified: 2007-10-20 17:16 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
/etc/init.d/apache2 with ${APACHE2_OPTS} added where missing (apache2,2.63 KB, text/plain)
2007-10-02 20:10 UTC, Mark J. Olah
Details
patch for /etc/init.d/apache2 (apache2_init_script.patch,898 bytes, patch)
2007-10-02 21:31 UTC, Mark J. Olah
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Mark J. Olah 2007-10-02 20:07:11 UTC
In www-servers/apache-2.2.6-r1, the init.d/apache2 script does not have ${APACHE2_OPTS} in the functions reload() graceful() gracefulstop().  I try to reload apache with: 
/etc/init.d/apache2 reload, I get errors because the defines from APACHE2_OPTS in conf.d/apache2 are not present and so ssl and other modules are not loaded.

Adding in ${APACHE2_OPTS} in these functions got rid of these errors for me.
I will attach the init.d/apache2 which works for me.  I'm not sure what the preferred method is to turn it directly into a patch, but if someone would let me know I could post the patch too.

Reproducible: Always

Steps to Reproduce:
1.Enable SSL option in conf.d
2.Have some SSL options in a config file without protective <IfDefine SSL> (I know this may be bad style, but the reload is not checking these portions of config files anyways, so if there was truely a bad directive in here there would be a problem)
3./etc/init.d/apache2
4./etc/init.d/apache2 reload
Comment 1 Mark J. Olah 2007-10-02 20:10:35 UTC
Created attachment 132419 [details]
/etc/init.d/apache2 with ${APACHE2_OPTS} added where missing
Comment 2 Jakub Moc (RETIRED) gentoo-dev 2007-10-02 21:16:33 UTC
Reopen with a unifies diff, thanks.
Comment 3 Mark J. Olah 2007-10-02 21:18:08 UTC
What does this mean?

(In reply to comment #2)
> Reopen with a unifies diff, thanks.
> 

Comment 4 Jakub Moc (RETIRED) gentoo-dev 2007-10-02 21:19:33 UTC
diff -Nau <old_file> <new_file>
Comment 5 Mark J. Olah 2007-10-02 21:31:12 UTC
Created attachment 132424 [details, diff]
patch for /etc/init.d/apache2
Comment 6 Mark J. Olah 2007-10-02 21:33:39 UTC
Reopening with patch.  I hope the filenames are okay in the patch, I wasn't sure what to call the old and new file so i named the distributed file apache2.dist and the updated file apache2.new  Let me know if there is a more canonical form.
Comment 7 Benedikt Böhm (RETIRED) gentoo-dev 2007-10-20 17:16:34 UTC
fixed in 2.2.6-r2