The extra xinit scripts are not essential for the functioning of the package, so I would put them in /usr/doc/ and put a note in README. Including them by default would be like including xinit scripts for using Xfwm4 with KDE and GNOME.<br>
<br><br><div><span class="gmail_quote">On 3/19/08, <b class="gmail_sendername">Chess Griffin</b> <<a href="mailto:chess@chessgriffin.com">chess@chessgriffin.com</a>> wrote:</span><blockquote class="gmail_quote" style="margin-top: 0; margin-right: 0; margin-bottom: 0; margin-left: 0; margin-left: 0.80ex; border-left-color: #cccccc; border-left-width: 1px; border-left-style: solid; padding-left: 1ex">
On Wed, 19 Mar 2008 17:57:04 -0700 (PDT)<br> Phillip Warner <<a href="mailto:pc_warner@yahoo.com">pc_warner@yahoo.com</a>> wrote:<br><br> > The xinitrc.openbox file included with the SlackBuild should be<br> > modified at the end so that it says: exec openbox-session<br>
><br> > instead of:<br> > exec openbox<br> ><br> > With the way it is, the autostart.sh file is not executed when<br> > starting openbox. This is important for setting up a stand alone<br> > openbox session. The file's output is slightly noisy out of the box,<br>
> however, as it does not redirect stderror and it checks for some<br> > things that slackware does not have out of the box. If this bothers<br> > people sed can fix that easily. This doesn't affect any<br>
> functionality.<br> ><br> > Also, since openbox includes scripts for starting KDE or GNOME<br> > sessions with openbox as the windows manager, it would be nice to<br> > have some xinitrc files for them as well. I propose that there be an<br>
> option to include each of their respective xinitrc scripts. They<br> > could be called xinitrc.kde-openbox and xinitrc.gnome-openbox, and<br> > the kde one could be included by default. The files would be the<br>
> same as the xinitrc.openbox file with the exception of the last line:<br> ><br> > exec openbox-kde-session<br> > - or -<br> > exec openbox-gnome-session<br> ><br> > I do not have GNOME, so I did not test the proposed xinitrc file for<br>
> it. However, I did test xinitrc.kde-openbox in both runlevels 3 and<br> > 4 and everything ran well.<br> ><br> > Having these xinitrc scripts will make it easier for the user to<br> > select them as their default session (with xwmconfig), and it will<br>
> make it so the sessions are an option in KDM/GDM/XDM.<br> ><br> > With these changes I think the openbox slackBuild would be improved<br> > and the resultant behavior would be more in line with the<br> > expectations based on the openbox documentation:<br>
> <a href="http://icculus.org/openbox/index.php/Help:Getting_started#Using_Openbox_without_a_desktop_environment_.28The_lightweight_approach.29">http://icculus.org/openbox/index.php/Help:Getting_started#Using_Openbox_without_a_desktop_environment_.28The_lightweight_approach.29</a><br>
><br> > --phil<br> ><br><br><br>Without having looked into this yet, I am inclined to agree with you on<br> the first point. As to the second point, it seems that maybe this is<br> more of a user-configurable option and not actually necessary for the<br>
stand-alone usage of Openbox? Couldn't the user do what you say<br> manually instead of crowding xwmconfig with a bunch of Openbox<br> choices? Or by option do you mean an option that can be enabled in the<br> SlackBuild? I don't use Openbox within KDE or GNOME so maybe I'm<br>
missing something. In any event, I'm just throwing out some thoughts.<br><br><br><br> --<br> Chess Griffin<br> GPG Public Key: 0x0C7558C3<br> <a href="http://www.chessgriffin.com">http://www.chessgriffin.com</a><br>
<br>_______________________________________________<br> Slackbuilds-users mailing list<br> <a href="mailto:Slackbuilds-users@slackbuilds.org">Slackbuilds-users@slackbuilds.org</a><br> <a href="http://lists.slackbuilds.org/mailman/listinfo/slackbuilds-users">http://lists.slackbuilds.org/mailman/listinfo/slackbuilds-users</a><br>
<br> Please read the FAQ - <a href="http://slackbuilds.org/faq/">http://slackbuilds.org/faq/</a><br><br><br></blockquote></div><br>