[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [E-devel] New patch for entranced and entrance
On Sun, 27 Aug 2006 23:22:38 +0300
Eugen Minciu <firstname.lastname@example.org> wrote:
And finally, I've got it.
The problem was in entrance_auth_setup_environment() which called clearenv().
For some reasong calling getenv() there did not return the path either. So I added an extra parameter, const char *path.
This finally works, I can run apps and everything appears to be working correctly.
> On Sun, 27 Aug 2006 22:36:31 +0300
> Eugen Minciu <email@example.com> wrote:
> I've figured out what causes the problem, but I don't know how to fix it this time.
> This happens because once I replace the execl() call in entrance_session_start_user_session, the environment variables for the user are no longer loaded. If anyone could tell me how to load these settings from C code, I'd very much appreciate it.
> As things stand, at this point, entrance does work with this, and the E17 session starts, but many issues can quickly be noticed, probably caused by not setting the environment variables to some proper values.
> > On Sun, 27 Aug 2006 22:30:58 +0300
> > Eugen Minciu <firstname.lastname@example.org> wrote:
> > Sorry, it seems that if I use this I get some issues within e. If I try to start an application from the run command thingie, it will tell me that the application failed to start.
> > I'll try to see what causes this. If anyone would be willing to investigate and/or give me any clues it would be appreciated.
> > > Here's a new patch for entranced and entrance.
> > >
> > > This time I did what the comment suggested in both places. I used strtok and parsed directly into a char ** which then gets run by execv/execvp.
> > >
> > > If my code strikes you as ugly (I'm not sure if it should) be sure to tell me what I did wrong and I'll try to fix it some more.
> > >
> > > BTW, this diff is made from e17/apps/entrance. I'm sorry, is this what you meant or should I have made it dirrectly from e17/ ?
> > >
> > > Eugen.
> > >