portage bug with summary.log
 

Hi Fabian,

Since we changed my EPREFIX to "" I've been getting summary.log
droppings in odd locations.

It turns out that the code in mod_save_summary.py has this...

if mysettings["PORT_LOGDIR"] != "":
elogdir = os.path.join(mysettings["PORT_LOGDIR"],
"elog")
else:
elogdir = os.path.join(EPREFIX, "var", "log", "portage",
"elog")

And because my EPREFIX is "" then we never get a "/" in front of var so
it creates var/log/portage/elog/summary.log in the place you emerge the
package.

We need a leading "/" here.

Can you fix ?

Alan.

portage bug with summary.log
 

On 10-03-2010 11:55:28 +0000, Alan Hourihane wrote:
> Hi Fabian,
>
> Since we changed my EPREFIX to "" I've been getting summary.log
> droppings in odd locations.
>
> It turns out that the code in mod_save_summary.py has this...
>
> if mysettings["PORT_LOGDIR"] != "":
> elogdir = os.path.join(mysettings["PORT_LOGDIR"],
> "elog")
> else:
> elogdir = os.path.join(EPREFIX, "var", "log", "portage",
> "elog")
>
> And because my EPREFIX is "" then we never get a "/" in front of var so
> it creates var/log/portage/elog/summary.log in the place you emerge the
> package.
>
> We need a leading "/" here.
>
> Can you fix ?

does this work for you?

elogdir = os.path.join("/", EPREFIX_LSTRIP, "var", "log", "portage", "elog");


--
Fabian Groffen
Gentoo on a different level

portage bug with summary.log
 

On Wed, 2010-03-10 at 13:01 +0100, Fabian Groffen wrote:
> On 10-03-2010 11:55:28 +0000, Alan Hourihane wrote:
> > Hi Fabian,
> >
> > Since we changed my EPREFIX to "" I've been getting summary.log
> > droppings in odd locations.
> >
> > It turns out that the code in mod_save_summary.py has this...
> >
> > if mysettings["PORT_LOGDIR"] != "":
> > elogdir = os.path.join(mysettings["PORT_LOGDIR"],
> > "elog")
> > else:
> > elogdir = os.path.join(EPREFIX, "var", "log", "portage",
> > "elog")
> >
> > And because my EPREFIX is "" then we never get a "/" in front of var so
> > it creates var/log/portage/elog/summary.log in the place you emerge the
> > package.
> >
> > We need a leading "/" here.
> >
> > Can you fix ?
>
> does this work for you?
>
> elogdir = os.path.join("/", EPREFIX_LSTRIP, "var", "log", "portage", "elog");

Yes.

Thanks!

Alan.