You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@avalon.apache.org by Peter Courcoux <pe...@courcoux.biz> on 2003/10/29 18:10:24 UTC
Merlin build
Hi, Steve, Alex,
The Merlin cvs build I checked out clean last night failed one of the
tests.
Test reports attached.
I think all the environment details are in the reports.
Regards,
Peter
--
Peter Courcoux <pe...@courcoux.biz>
RE: Merlin build
Posted by Peter Courcoux <pe...@courcoux.biz>.
Thanks Alex.
On Wed, 2003-10-29 at 21:17, Alex Karasulu wrote:
> Steve,
>
> Here's a patch to Env in
> merlen/kernel/loader/src/java/org/apache/avalon/merlin that basically forks
> a shell to expand out environment macros instead of forking 'echo' alone.
> It btw checks to see what the ${user.name} default shell is to determine
> which shell in UNIX to fork. The problem was UNIX specific but it is done
> now. I tested it on both Solaris and Linux to make sure.
>
> Alex
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@avalon.apache.org
> For additional commands, e-mail: dev-help@avalon.apache.org
--
Peter Courcoux <pe...@courcoux.biz>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@avalon.apache.org
For additional commands, e-mail: dev-help@avalon.apache.org
Re: Merlin build
Posted by Stephen McConnell <mc...@apache.org>.
Applied.
Alex Karasulu wrote:
>Can't seem to get anything write today. Here's the patch on the right email
>trail.
>
>Shooooh!
>
>Alex
>
>
>
>>-----Original Message-----
>>From: Alex Karasulu [mailto:aok123@bellsouth.net]
>>Sent: Wednesday, October 29, 2003 4:17 PM
>>To: 'Avalon Developers List'
>>Subject: RE: Merlin build
>>
>>Steve,
>>
>>Here's a patch to Env in
>>merlen/kernel/loader/src/java/org/apache/avalon/merlin that basically
>>forks
>>a shell to expand out environment macros instead of forking 'echo' alone.
>>It btw checks to see what the ${user.name} default shell is to determine
>>which shell in UNIX to fork. The problem was UNIX specific but it is done
>>now. I tested it on both Solaris and Linux to make sure.
>>
>>Alex
>>
>>
>>
>>---------------------------------------------------------------------
>>To unsubscribe, e-mail: dev-unsubscribe@avalon.apache.org
>>For additional commands, e-mail: dev-help@avalon.apache.org
>>
>>
>
>
>
>
>------------------------------------------------------------------------
>
>---------------------------------------------------------------------
>To unsubscribe, e-mail: dev-unsubscribe@avalon.apache.org
>For additional commands, e-mail: dev-help@avalon.apache.org
>
--
Stephen J. McConnell
mailto:mcconnell@apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@avalon.apache.org
For additional commands, e-mail: dev-help@avalon.apache.org
Re: Merlin build
Posted by Stephen McConnell <mc...@apache.org>.
Its' not making it thought.
Can you send it to me directly?
Steve.
Alex Karasulu wrote:
>Can't seem to get anything write today. Here's the patch on the right email
>trail.
>
>Shooooh!
>
>Alex
>
>
>
>>-----Original Message-----
>>From: Alex Karasulu [mailto:aok123@bellsouth.net]
>>Sent: Wednesday, October 29, 2003 4:17 PM
>>To: 'Avalon Developers List'
>>Subject: RE: Merlin build
>>
>>Steve,
>>
>>Here's a patch to Env in
>>merlen/kernel/loader/src/java/org/apache/avalon/merlin that basically
>>forks
>>a shell to expand out environment macros instead of forking 'echo' alone.
>>It btw checks to see what the ${user.name} default shell is to determine
>>which shell in UNIX to fork. The problem was UNIX specific but it is done
>>now. I tested it on both Solaris and Linux to make sure.
>>
>>Alex
>>
>>
>>
>>---------------------------------------------------------------------
>>To unsubscribe, e-mail: dev-unsubscribe@avalon.apache.org
>>For additional commands, e-mail: dev-help@avalon.apache.org
>>
>>
>
>
>
>
>------------------------------------------------------------------------
>
>---------------------------------------------------------------------
>To unsubscribe, e-mail: dev-unsubscribe@avalon.apache.org
>For additional commands, e-mail: dev-help@avalon.apache.org
>
--
Stephen J. McConnell
mailto:mcconnell@apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@avalon.apache.org
For additional commands, e-mail: dev-help@avalon.apache.org
RE: Merlin build
Posted by Alex Karasulu <ao...@bellsouth.net>.
Can't seem to get anything write today. Here's the patch on the right email
trail.
Shooooh!
Alex
> -----Original Message-----
> From: Alex Karasulu [mailto:aok123@bellsouth.net]
> Sent: Wednesday, October 29, 2003 4:17 PM
> To: 'Avalon Developers List'
> Subject: RE: Merlin build
>
> Steve,
>
> Here's a patch to Env in
> merlen/kernel/loader/src/java/org/apache/avalon/merlin that basically
> forks
> a shell to expand out environment macros instead of forking 'echo' alone.
> It btw checks to see what the ${user.name} default shell is to determine
> which shell in UNIX to fork. The problem was UNIX specific but it is done
> now. I tested it on both Solaris and Linux to make sure.
>
> Alex
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@avalon.apache.org
> For additional commands, e-mail: dev-help@avalon.apache.org
RE: Merlin build
Posted by Alex Karasulu <ao...@bellsouth.net>.
Steve,
Here's a patch to Env in
merlen/kernel/loader/src/java/org/apache/avalon/merlin that basically forks
a shell to expand out environment macros instead of forking 'echo' alone.
It btw checks to see what the ${user.name} default shell is to determine
which shell in UNIX to fork. The problem was UNIX specific but it is done
now. I tested it on both Solaris and Linux to make sure.
Alex
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@avalon.apache.org
For additional commands, e-mail: dev-help@avalon.apache.org
RE: Merlin build
Posted by Peter Courcoux <pe...@courcoux.biz>.
On Wed, 2003-10-29 at 19:52, Alex Karasulu wrote:
> Peter,
>
> Yeah the problem is that shell based macro expansion is not working when you
> do a simple 'echo $PATH' expression either: the echo takes it literally and
> returns it to you since it is not in an interpreted environment (a shell).
> You need to fork a shell and have it run the evaluation. Also the user's
> shell must be determined so the user's /etc/passwd entry must be read to get
> the default shell. Getting more involved than I wanted to ...
>
> Even this might not be the shell being used but there is no other way to
> determine what stinking shell was running when the runtime was started.
>
> If we don't pick the default shell, the environment variable may not be
> present. Yeah this stuff wreaks pretty bad and is the pits. That's why
> people stay clear of it. I'll have an update for you as soon as I have it
> working properly and tested on a couple of platforms.
>
> Sorry about that,
> Alex
No problem. It was getting late when I hit this last evening. I'm glad I
didn't sit up trying to fix it ;-)
Peter
>
> > > Meaning comment out line 52 where the test asserts a non-null for the
> > PATH
> > > variable.
> >
> > I think the actual error is that getUnixVariable("$PAT") is returning
> > "$PAT" at line 51. I added some System.out.println 's.
> >
> > Regards,
> >
> > Peter
> >
> > >
> > >
> > > > -----Original Message-----
> > > > From: Alex Karasulu [mailto:aok123@bellsouth.net]
> > > > Sent: Wednesday, October 29, 2003 12:19 PM
> > > > To: 'Avalon Developers List'
> > > > Subject: RE: Merlin build
> > > >
> > > > Interesting because the TestCase can't seem to find a PATH variable in
> > > > your
> > > > environment when it should. For the time being try to comment it
> > out. I
> > > > will try it on a linux box and get back to you shortly.
> > > >
> > > > Alex
> > > >
> > > > > -----Original Message-----
> > > > > From: Peter Courcoux [mailto:peter@courcoux.biz]
> > > > > Sent: Wednesday, October 29, 2003 12:10 PM
> > > > > To: Avalon Dev List
> > > > > Subject: Merlin build
> > > > >
> > > > > Hi, Steve, Alex,
> > > > >
> > > > > The Merlin cvs build I checked out clean last night failed one of
> > the
> > > > > tests.
> > > > >
> > > > > Test reports attached.
> > > > >
> > > > > I think all the environment details are in the reports.
> > > > >
> > > > > Regards,
> > > > >
> > > > > Peter
> > > > > --
> > > > > Peter Courcoux <pe...@courcoux.biz>
> > > >
> > > >
> > > >
> > > > ---------------------------------------------------------------------
> > > > To unsubscribe, e-mail: dev-unsubscribe@avalon.apache.org
> > > > For additional commands, e-mail: dev-help@avalon.apache.org
> > >
> > >
> > >
> > >
> > > ---------------------------------------------------------------------
> > > To unsubscribe, e-mail: dev-unsubscribe@avalon.apache.org
> > > For additional commands, e-mail: dev-help@avalon.apache.org
> > --
> > Peter Courcoux <pe...@courcoux.biz>
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: dev-unsubscribe@avalon.apache.org
> > For additional commands, e-mail: dev-help@avalon.apache.org
>
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@avalon.apache.org
> For additional commands, e-mail: dev-help@avalon.apache.org
--
Peter Courcoux <pe...@courcoux.biz>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@avalon.apache.org
For additional commands, e-mail: dev-help@avalon.apache.org
RE: Merlin build
Posted by Alex Karasulu <ao...@bellsouth.net>.
Peter,
Yeah the problem is that shell based macro expansion is not working when you
do a simple 'echo $PATH' expression either: the echo takes it literally and
returns it to you since it is not in an interpreted environment (a shell).
You need to fork a shell and have it run the evaluation. Also the user's
shell must be determined so the user's /etc/passwd entry must be read to get
the default shell. Getting more involved than I wanted to ...
Even this might not be the shell being used but there is no other way to
determine what stinking shell was running when the runtime was started.
If we don't pick the default shell, the environment variable may not be
present. Yeah this stuff wreaks pretty bad and is the pits. That's why
people stay clear of it. I'll have an update for you as soon as I have it
working properly and tested on a couple of platforms.
Sorry about that,
Alex
> > Meaning comment out line 52 where the test asserts a non-null for the
> PATH
> > variable.
>
> I think the actual error is that getUnixVariable("$PAT") is returning
> "$PAT" at line 51. I added some System.out.println 's.
>
> Regards,
>
> Peter
>
> >
> >
> > > -----Original Message-----
> > > From: Alex Karasulu [mailto:aok123@bellsouth.net]
> > > Sent: Wednesday, October 29, 2003 12:19 PM
> > > To: 'Avalon Developers List'
> > > Subject: RE: Merlin build
> > >
> > > Interesting because the TestCase can't seem to find a PATH variable in
> > > your
> > > environment when it should. For the time being try to comment it
> out. I
> > > will try it on a linux box and get back to you shortly.
> > >
> > > Alex
> > >
> > > > -----Original Message-----
> > > > From: Peter Courcoux [mailto:peter@courcoux.biz]
> > > > Sent: Wednesday, October 29, 2003 12:10 PM
> > > > To: Avalon Dev List
> > > > Subject: Merlin build
> > > >
> > > > Hi, Steve, Alex,
> > > >
> > > > The Merlin cvs build I checked out clean last night failed one of
> the
> > > > tests.
> > > >
> > > > Test reports attached.
> > > >
> > > > I think all the environment details are in the reports.
> > > >
> > > > Regards,
> > > >
> > > > Peter
> > > > --
> > > > Peter Courcoux <pe...@courcoux.biz>
> > >
> > >
> > >
> > > ---------------------------------------------------------------------
> > > To unsubscribe, e-mail: dev-unsubscribe@avalon.apache.org
> > > For additional commands, e-mail: dev-help@avalon.apache.org
> >
> >
> >
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: dev-unsubscribe@avalon.apache.org
> > For additional commands, e-mail: dev-help@avalon.apache.org
> --
> Peter Courcoux <pe...@courcoux.biz>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@avalon.apache.org
> For additional commands, e-mail: dev-help@avalon.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@avalon.apache.org
For additional commands, e-mail: dev-help@avalon.apache.org
RE: Merlin build
Posted by Peter Courcoux <pe...@courcoux.biz>.
On Wed, 2003-10-29 at 17:25, Alex Karasulu wrote:
> Meaning comment out line 52 where the test asserts a non-null for the PATH
> variable.
I think the actual error is that getUnixVariable("$PAT") is returning
"$PAT" at line 51. I added some System.out.println 's.
Regards,
Peter
>
>
> > -----Original Message-----
> > From: Alex Karasulu [mailto:aok123@bellsouth.net]
> > Sent: Wednesday, October 29, 2003 12:19 PM
> > To: 'Avalon Developers List'
> > Subject: RE: Merlin build
> >
> > Interesting because the TestCase can't seem to find a PATH variable in
> > your
> > environment when it should. For the time being try to comment it out. I
> > will try it on a linux box and get back to you shortly.
> >
> > Alex
> >
> > > -----Original Message-----
> > > From: Peter Courcoux [mailto:peter@courcoux.biz]
> > > Sent: Wednesday, October 29, 2003 12:10 PM
> > > To: Avalon Dev List
> > > Subject: Merlin build
> > >
> > > Hi, Steve, Alex,
> > >
> > > The Merlin cvs build I checked out clean last night failed one of the
> > > tests.
> > >
> > > Test reports attached.
> > >
> > > I think all the environment details are in the reports.
> > >
> > > Regards,
> > >
> > > Peter
> > > --
> > > Peter Courcoux <pe...@courcoux.biz>
> >
> >
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: dev-unsubscribe@avalon.apache.org
> > For additional commands, e-mail: dev-help@avalon.apache.org
>
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@avalon.apache.org
> For additional commands, e-mail: dev-help@avalon.apache.org
--
Peter Courcoux <pe...@courcoux.biz>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@avalon.apache.org
For additional commands, e-mail: dev-help@avalon.apache.org
RE: Merlin build
Posted by Alex Karasulu <ao...@bellsouth.net>.
Meaning comment out line 52 where the test asserts a non-null for the PATH
variable.
> -----Original Message-----
> From: Alex Karasulu [mailto:aok123@bellsouth.net]
> Sent: Wednesday, October 29, 2003 12:19 PM
> To: 'Avalon Developers List'
> Subject: RE: Merlin build
>
> Interesting because the TestCase can't seem to find a PATH variable in
> your
> environment when it should. For the time being try to comment it out. I
> will try it on a linux box and get back to you shortly.
>
> Alex
>
> > -----Original Message-----
> > From: Peter Courcoux [mailto:peter@courcoux.biz]
> > Sent: Wednesday, October 29, 2003 12:10 PM
> > To: Avalon Dev List
> > Subject: Merlin build
> >
> > Hi, Steve, Alex,
> >
> > The Merlin cvs build I checked out clean last night failed one of the
> > tests.
> >
> > Test reports attached.
> >
> > I think all the environment details are in the reports.
> >
> > Regards,
> >
> > Peter
> > --
> > Peter Courcoux <pe...@courcoux.biz>
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@avalon.apache.org
> For additional commands, e-mail: dev-help@avalon.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@avalon.apache.org
For additional commands, e-mail: dev-help@avalon.apache.org
RE: Merlin build
Posted by Alex Karasulu <ao...@bellsouth.net>.
Interesting because the TestCase can't seem to find a PATH variable in your
environment when it should. For the time being try to comment it out. I
will try it on a linux box and get back to you shortly.
Alex
> -----Original Message-----
> From: Peter Courcoux [mailto:peter@courcoux.biz]
> Sent: Wednesday, October 29, 2003 12:10 PM
> To: Avalon Dev List
> Subject: Merlin build
>
> Hi, Steve, Alex,
>
> The Merlin cvs build I checked out clean last night failed one of the
> tests.
>
> Test reports attached.
>
> I think all the environment details are in the reports.
>
> Regards,
>
> Peter
> --
> Peter Courcoux <pe...@courcoux.biz>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@avalon.apache.org
For additional commands, e-mail: dev-help@avalon.apache.org