Closed Bug 230468 Opened 21 years ago Closed 21 years ago

RFE: mozilla should provide a simple way to run custom shell scripts at mozilla startup and shutdown

Categories

(SeaMonkey :: Build Config, enhancement)

All
Linux
enhancement
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: roland.mainz, Assigned: roland.mainz)

References

Details

Attachments

(1 file, 1 obsolete file)

RFE: mozilla should provide a simple way to run custom shell scripts at mozilla
startup and shutdown

Possible purposes:
- Set/Unset font paths
- Set additional environment variables without modifying the mozilla scripts
directly
- Start and stop additional applications/shell scripts before mozilla startup
and after shutdown

Design:
The new functionality should work like this:
- Placing a script "S01" into ${MOZILLA_FIVE_HOME}/init.d/ cases the script to
be executed at Mozilla startup.
- Placing a script "K01" into ${MOZILLA_FIVE_HOME}/init.d/ cases the script to
be executed at Mozilla shutdown.
- Startup/shutdown scripts ending with *.sh will be executed "inline" as part of
"run-mozilla.sh", being able to set/alter existing env variables to fix the
needs of the user.
Status: NEW → ASSIGNED
Attached patch Patch for 2004-01-07-trunk (obsolete) — Splinter Review
Note that this new patch does not affect the GRE setup in any way so the
"political" problems ("no scripts used by GRE. Never. Ever!") discussed in bug
226803 do not apply here.
Comment on attachment 138678 [details] [diff] [review]
Patch for 2004-01-07-trunk

Requesting r= ...
Attachment #138678 - Flags: review?(leaf)
The new patch allows that other consumers of the script can set their own value
for MOZ_USER_DIR and run-mozilla.sh is now called inline, saving a few CPU
cycles.
Attachment #138678 - Attachment is obsolete: true
Attachment #138681 - Flags: review?(leaf)
Attachment #138678 - Flags: review?(leaf)
Comment on attachment 138681 [details] [diff] [review]
Better patch for 2004-01-07-trunk

ok, I'm convinced this is OK for the seamonkey client. Please don't check this
in without MOA=leaf.
Attachment #138681 - Flags: review?(leaf) → review+
Comment on attachment 138681 [details] [diff] [review]
Better patch for 2004-01-07-trunk

Requesting moa=leaf ...
Attachment #138681 - Flags: superreview?(leaf)
Comment on attachment 138681 [details] [diff] [review]
Better patch for 2004-01-07-trunk

leaf: ping!
Comment on attachment 138681 [details] [diff] [review]
Better patch for 2004-01-07-trunk

super-rheeet=leaf
Attachment #138681 - Flags: superreview?(leaf) → superreview+
Checking in Makefile.in;
/cvsroot/mozilla/xpfe/bootstrap/Makefile.in,v  <--  Makefile.in
new revision: 1.248; previous revision: 1.247
done
Checking in mozilla.in;
/cvsroot/mozilla/xpfe/bootstrap/mozilla.in,v  <--  mozilla.in
new revision: 1.4; previous revision: 1.3
done
Status: ASSIGNED → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
could this have caused Bug 233014, starting BuildID 1.7.0.2004020317 ?
Bug 233014 default install into windows\program files directory, erases files 
has this patch been tested under windows ?
Hermann Schwab wrote:
> could this have caused Bug 233014, starting BuildID 1.7.0.2004020317 ?
> Bug 233014 default install into windows\program files directory, erases files

How should this patch affect windows ? The script is AFAIK only used on
Unix/Linux ...
Bernd wrote:
> has this patch been tested under windows ?

No... the only thing I did was to search the windows ZIP archives whether the
script is present in these tarballs. Since there were no hit I assumed Windows
is not affected by this change.
When using 'make install' to install a build, I wind up with these extra init.d
dirs under /usr/local/bin/ .  Since those are mozilla specific, they should be
placed under the $mozappdir not $bindir .
cls wrote:
> When using 'make install' to install a build, I wind up with these extra 
> init.d dirs under /usr/local/bin/ .  Since those are mozilla specific, they 
> should be placed under the $mozappdir not $bindir

OK; I'll fix that as part of bug 234124 ("RFE: Port the pluggable shell script
API to FireFox/ThunderBird/SunBird etc") ...
Blocks: 234124
Blocks: 238360
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: