[Slackbuilds-users] Fwd: bug on openbox package

Tue Jul 8 05:47:34 UTC 2008

Attached is the diff for the slackBuild.  xinitrc.openbox-session (not shown in the diff) is the same as the old xinitrc.openbox file.

I tested the new script and everything seems to work great.  You can easily see the difference between running openbox and openbox-session on a default profile because with the later the autostart file causes the background to change (the other will just leave the root image the same as in the KDM screen).

It also might be a good idea to add better config file support in the doinst.sh file for the files under /etc/xdg/openbox.  That way upgrading openbox won't necessarily overwrite your default menu and autostart.sh file.  They will create .new files instead.  There are plenty of examples of this in other slackBuild scripts.

--phil

--- On Sun, 7/6/08, Phillip Warner <pc_warner at yahoo.com> wrote:

> From: Phillip Warner <pc_warner at yahoo.com>
> Subject: Re: [Slackbuilds-users] Fwd: bug on openbox package
> To: "Chess Griffin" <chess at chessgriffin.com>, "SlackBuilds.org Users List" <slackbuilds-users at slackbuilds.org>
> Date: Sunday, July 6, 2008, 9:11 PM
> I should have time to do it in the next day or two (but not
> right now), and it shouldn't take me long at all.  You
> have the right idea of what needs to be done, but you mixed
> up the files names a little.  Here is basically what needs
> to be done:
> 
> *) make a copy of xinitrc.openbox to
> xinitrc.openbox-session and change the original by removing
> the '-session' from the last line (make sure the
> second xinitrc gets installed into /etc/X11/xinit/ as
> well).  You don't need to copy the file in the script;
> just having two xinitrc's in the slackBuild folder will
> work.
> *) mv /usr/share/xsessions/openbox.desktop
> /usr/share/xsessions/openbox-session.desktop
> *) sed the file to change 'Name' to Openbox-Session
> (to differentiate from the other kdm openbox option)
> *) Create symlinks under /usr/share/apps/kdm/sessions/ for
> the openbox*.desktop files (openbox-kde.desktop,
> openbox-gnome.desktop, openbox-session.desktop) under
> /usr/share/xsessions/
> 
> Yes, as it is currently (without the fixes), the Openbox
> choice in kdm will execute openbox instead of
> openbox-session.  As the Openbox docs say, you want to use
> the openbox-session if you want to make use of the
> autostart file and session management features.  If I
> remember correctly one place that information can be found
> is on
> http://icculus.org/openbox/index.php/Help:Getting_started. 
> A note should probably be in the slackBuild README about
> this.
> 
> I'll check my email tomorrow evening before updating
> the script.  If you already have it taken care of just let
> me know.
> 
> --phil
> 
> --- On Sat, 7/5/08, Chess Griffin
> <chess at chessgriffin.com> wrote:
> 
> > From: Chess Griffin <chess at chessgriffin.com>
> > Subject: Re: [Slackbuilds-users] Fwd: bug on openbox
> package
> > To: pc_warner at yahoo.com, "SlackBuilds.org Users
> List" <slackbuilds-users at slackbuilds.org>
> > Date: Saturday, July 5, 2008, 8:07 PM
> > On Sat, Jul 5, 2008 at 5:20 PM, Phillip Warner
> > <pc_warner at yahoo.com> wrote:
> > > Yes, the stock openbox.desktop file from the
> kdebase
> > package should execute openbox-session.  However, the
> > problem is more than that.  The new kdm doesn't
> seem to
> > make use of the .desktop files under
> /usr/share/xsessions/,
> > which is where Openbox puts them by default.  Thus, in
> > order for the Slackware 12.1 Openbox slackBuild to
> work
> > properly with KDM, the openbox*.desktop files under
> there
> > must either be moved or have a corresponding symlink
> in
> > /usr/share/apps/kdm/sessions/ (the latter is probably
> > better in case someone is using something other than
> kdm). 
> > The default location was fine in the kdm in Slackware
> 12.0.
> > >
> > 
> > Phil,
> > 
> > Thanks for your very helpful explanation.  I do see
> the
> > issue and
> > agree with your proposed solutions.  I also think we
> > definitely do not
> > want to overwrite any stock Slackware files and at the
> same
> > time
> > preserve the ability to use the SlackBuild for those
> that
> > do not use
> > KDE or KDM.  I also like your suggestion to have both
> the
> > xinitrc.openbox-session and xinitrc.openbox setup as
> > originally
> > intended by the Openbox developers.  If I understand
> your
> > solution
> > correctly, we would need to (1) create a new
> > xinitrc-session.desktop
> > file and install it into the default Openbox location
> along
> > with the
> > other Openbox *desktop files (2) symlink all the
> Openbox
> > *desktop
> > files including the new xinitrc-session.desktop file
> to the
> > KDM
> > location and (3) create a new xinitrc.openbox-session
> file
> > along with
> > the current xinitrc.openbox that already comes with
> the
> > SlackBuild?
> > Since we already include a xinitrc.openbox file in the
> > SlackBuild, I
> > assume then the stock openbox.desktop file works ok
> for
> > someone who
> > does not want to launch an Openbox session but just
> > straight Openbox?
> > In any event, yes, if you have the time to send your
> > modifications to
> > the SlackBuild and related files that would be great. 
> > Otherwise, I'll
> > try and work on this during the coming week.
> > 
> > In the meantime, I'll install KDE and KDM on one
> of my
> > Slack machines
> > so I can test and get a better feel for what KDM is
> doing
> > behind the
> > scenes.  I've never been a graphical login kind of
> > person, which is
> > why it's taken me longer than normal to get the
> issue
> > here.  It has
> > nothing to do with the fact that I'm about to turn
> 40
> > or the loss of
> > brain cells from one too many pints of Guinness in my
> > youth.  At
> > least, that's my story and I'm sticking to it.
> > 
> > Thanks again for your help, Phil, and to Bruno for the
> > original bug report.
> > 
> > -- 
> > Chess Griffin
> > GPG Public Key: 0x0C7558C3
> > http://www.chessgriffin.com
> 
> 
>       
> _______________________________________________
> SlackBuilds-users mailing list
> SlackBuilds-users at slackbuilds.org
> http://lists.slackbuilds.org/mailman/listinfo/slackbuilds-users
> Archives -
> http://lists.slackbuilds.org/pipermail/slackbuilds-users/
> FAQ - http://slackbuilds.org/faq/

-------------- next part --------------
A non-text attachment was scrubbed...
Name: openbox-slackBuild-2.diff
Type: application/octet-stream
Size: 3861 bytes
Desc: not available
URL: <http://lists.slackbuilds.org/pipermail/slackbuilds-users/attachments/20080707/b7cf0fd7/attachment.obj>