Re: [E-devel] entrance autodetect.sh integration.

2006-03-29 Thread littletux
Hello,

Le mercredi 08 mars 2006 à 17:03 -0500, Michael Jennings a écrit :
> > The autodetect.sh script tries to figure out the proper vt to use
> > for the currently running system, so it probably should not be run
> > by package builders.
> 
> If it's part of "make install" is WILL BE RUN by package builders. 
Unless you add something like this before running configure script :
perl -pi -e "s|sh data/config/autodetect.sh|#sh \
data/config/autodetect.sh|" Makefile.in

If there was a configure option to (enable or) disable the autodetect.sh
script, it could also be a (cleaner) solution.

Best regards,

Guillaume B.





---
This SF.Net email is sponsored by xPML, a groundbreaking scripting language
that extends applications into web and mobile media. Attend the live webcast
and join the prime developer group breaking into this new coding territory!
http://sel.as-us.falkag.net/sel?cmd=lnk&kid0944&bid$1720&dat1642
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] entrance autodetect.sh integration.

2006-03-08 Thread Michael Jennings
On Thursday, 09 March 2006, at 07:54:38 (+1000),
David Seikel wrote:

> I have not tried it from a package builders point of view.

Then perhaps you should.  Or at least try to comprehend such a point
of view before you change things of which you may have an inaccurate
or incomplete understanding.

> The autodetect.sh script tries to figure out the proper vt to use
> for the currently running system, so it probably should not be run
> by package builders.

If it's part of "make install" is WILL BE RUN by package builders.

> They should probably choose a vt and configure that manually.  The
> pam part just wants to clone a pam config file from whatever other
> display manager is currently installed at the time, so the same
> applies.

Neither of these should be run as part of "make install."  They should
only happen as a result of an individual user's choice when
installing.

> I'll be the first to admit that my autofoo fu is non existent, I had
> a hard enough time coming up with what I eventually committed.
> Sorting it out for package builders might be beyond me current
> skills.

Then leave what was already there.

Michael

-- 
Michael Jennings (a.k.a. KainX)  http://www.kainx.org/  <[EMAIL PROTECTED]>
n + 1, Inc., http://www.nplus1.net/   Author, Eterm (www.eterm.org)
---
 "A good name is more desirable than great riches; to be esteemed is
  better than silver or gold.  Rich and poor have this in common:
  the Lord is Maker of them all."   -- Proverbs 22:1-2


---
This SF.Net email is sponsored by xPML, a groundbreaking scripting language
that extends applications into web and mobile media. Attend the live webcast
and join the prime developer group breaking into this new coding territory!
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=110944&bid=241720&dat=121642
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] entrance autodetect.sh integration.

2006-03-08 Thread David Seikel
On Wed, 08 Mar 2006 17:57:20 +0100 Morten Nilsen <[EMAIL PROTECTED]>
wrote:

> David Seikel wrote:
> > On Tue, 28 Feb 2006 18:30:35 +1000 David Seikel <[EMAIL PROTECTED]>
> > wrote:
> > 
> >> I say we should add autodetect.sh to the things that are done
> >> during make install.  I will do so after a suitable period for
> >> discussion unless someone objects, or someone beats me to it.
> > 
> > It's been a week, and no one complained.  I'll do it tonight, that's
> > sure to bring the complaints out of the wood work.
> 
> let me just chime in now then :) took a look at the script, and I
> have one question and on objection..
> 
> -- when building packages, will the script change 
> /etc/entrance_config.cfg, or will it attempt copying 
> /var/tmp/entrance-build-root/etc/pam.d/* ?
> 
> in either case, one of these things will break.

I have not tried it from a package builders point of view.  The
autodetect.sh script tries to figure out the proper vt to use for the
currently running system, so it probably should not be run by package
builders.  They should probably choose a vt and configure that
manually.  The pam part just wants to clone a pam config file from
whatever other display manager is currently installed at the time, so
the same applies.

I'll be the first to admit that my autofoo fu is non existent, I had a
hard enough time coming up with what I eventually committed.  Sorting
it out for package builders might be beyond me current skills.

> -- sudo? why? from a packaging standpoint, needing sudo properly set
> up to build the package is plain silly.

sudo is just left over from before it was integrated into make
install.  For obvious reasons, and one not obvious reason, it needs to
do some things as root.  The not so obvious reason is that the
invocation of lsof on some systems needs to be done as root, at least
that particular invocation does.  Before the integration into make
install, it was most likely run by a non root user, hence the need for
sudo.  Now that it is part of make install, it will be run as root
anyway, so we can get rid of sudo.  I hadn't changed the script yet.


pgpVUXXmQr5ji.pgp
Description: PGP signature


Re: [E-devel] entrance autodetect.sh integration.

2006-03-08 Thread Michael Jennings
On Wednesday, 08 March 2006, at 17:57:20 (+0100),
Morten Nilsen wrote:

> -- sudo? why? from a packaging standpoint, needing sudo properly set up 
> to build the package is plain silly.

Any requirement of sudo in any form is inappropriate.

Michael

-- 
Michael Jennings (a.k.a. KainX)  http://www.kainx.org/  <[EMAIL PROTECTED]>
n + 1, Inc., http://www.nplus1.net/   Author, Eterm (www.eterm.org)
---
 "To turn away and not become another nail to pierce the skin of One
  who loved more deeply than the oceans, more abundant than the tears
  of a world embracing every heartache."
   -- Jars of Clay, "Worlds Apart"


---
This SF.Net email is sponsored by xPML, a groundbreaking scripting language
that extends applications into web and mobile media. Attend the live webcast
and join the prime developer group breaking into this new coding territory!
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=110944&bid=241720&dat=121642
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] entrance autodetect.sh integration.

2006-03-08 Thread Morten Nilsen

David Seikel wrote:

On Tue, 28 Feb 2006 18:30:35 +1000 David Seikel <[EMAIL PROTECTED]>
wrote:


I say we should add autodetect.sh to the things that are done during
make install.  I will do so after a suitable period for discussion
unless someone objects, or someone beats me to it.


It's been a week, and no one complained.  I'll do it tonight, that's
sure to bring the complaints out of the wood work.


let me just chime in now then :) took a look at the script, and I have 
one question and on objection..


-- when building packages, will the script change 
/etc/entrance_config.cfg, or will it attempt copying 
/var/tmp/entrance-build-root/etc/pam.d/* ?


in either case, one of these things will break.

-- sudo? why? from a packaging standpoint, needing sudo properly set up 
to build the package is plain silly.


Cheers,
--
Morten
:wq


---
This SF.Net email is sponsored by xPML, a groundbreaking scripting language
that extends applications into web and mobile media. Attend the live webcast
and join the prime developer group breaking into this new coding territory!
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=110944&bid=241720&dat=121642
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] entrance autodetect.sh integration.

2006-03-07 Thread David Seikel
On Tue, 28 Feb 2006 18:30:35 +1000 David Seikel <[EMAIL PROTECTED]>
wrote:

> I say we should add autodetect.sh to the things that are done during
> make install.  I will do so after a suitable period for discussion
> unless someone objects, or someone beats me to it.

It's been a week, and no one complained.  I'll do it tonight, that's
sure to bring the complaints out of the wood work.


pgpZybySix8jx.pgp
Description: PGP signature


[E-devel] entrance autodetect.sh integration.

2006-02-28 Thread David Seikel
I feel it's time to integrate
e17/apps/entrance/data/config/autodetect.sh into the install process.
It's been there for a while, people are using it, no one has
complained that it doesn't work.  Whenever anybody complains about the
keyboard freezing problem, I suggest they run that script, they do,
and it always fixes it.

make install will overwrite whatever entrance_config.cfg file the system
has with the default one, integrating autodetect.sh into make install
will at least make sure that it has the vt and pam stuff setup
correctly.

I say we should add autodetect.sh to the things that are done during
make install.  I will do so after a suitable period for discussion
unless someone objects, or someone beats me to it.

While on the subject, wasn't someone writing sessions detection code to
generate the entrance sessions list?  If that is not happening, I'll
add it to my TODO, as I think fdo have something to say about that, and
I can do it after I have finished dealing with fdo menus.


pgpKbzow5rRTX.pgp
Description: PGP signature