| 1 |
On Sun, 14 Apr 2013 12:25:27 +0200 |
| 2 |
Pacho Ramos <pacho@g.o> wrote: |
| 3 |
|
| 4 |
> El dom, 14-04-2013 a las 12:08 +0200, Michał Górny escribió: |
| 5 |
> > On Sun, 14 Apr 2013 11:59:14 +0200 |
| 6 |
> > Pacho Ramos <pacho@g.o> wrote: |
| 7 |
> > |
| 8 |
> > > El dom, 14-04-2013 a las 11:45 +0200, Michał Górny escribió: |
| 9 |
> > > > On Sun, 14 Apr 2013 11:40:03 +0200 |
| 10 |
> > > > Pacho Ramos <pacho@g.o> wrote: |
| 11 |
> > > > |
| 12 |
> > > > > # >=mono-0.92 versions using mcs -pkg:foo-sharp require shared memory, so we set the |
| 13 |
> > > > > # shared dir to ${T} so that ${T}/.wapi can be used during the install process. |
| 14 |
> > > > > export MONO_SHARED_DIR="${T}" |
| 15 |
> > > > |
| 16 |
> > > > Don't use ${T} in global scope. And just don't export them |
| 17 |
> > > > in the global scope either. |
| 18 |
> > > |
| 19 |
> > > Why not? |
| 20 |
> > |
| 21 |
> > Let's start with the fact that ${T} is only partially persistent |
| 22 |
> > by the words of PMS. I don't know if it's really relevant here but |
| 23 |
> > you're exporting persistent variables with value based on |
| 24 |
> > an non-persistent one. |
| 25 |
> > |
| 26 |
> > Thinking about it more, it probably would work. As long as you don't |
| 27 |
> > assume anything about those directories on pkg_*rm() where ${T} would |
| 28 |
> > have changed already and your variables wouldn't. |
| 29 |
> > |
| 30 |
> |
| 31 |
> Yes, they will be needed at compile time, that would explain why no |
| 32 |
> problem raised for now :/ Thanks for the info |
| 33 |
|
| 34 |
Well, I am a bit worried about metadata regen but I guess even having |
| 35 |
invalid values there doesn't hurt anyone. Well, assuming that the |
| 36 |
eclass will always be sourced before running the build and noone will |
| 37 |
try to use a pre-cached environment. |
| 38 |
|
| 39 |
> > Also, why are you exporting HOME? PMS does that already... |
| 40 |
> > |
| 41 |
> |
| 42 |
> Probably because it's inherited from current mono.eclass, but, are you |
| 43 |
> sure PMS does that already? There are more examples in the tree (in |
| 44 |
> eclasses and ebuilds) exporting HOME in similar way (vim.eclass for |
| 45 |
> example) :/ Or maybe it was started to be exported more recently and |
| 46 |
> this is only a relic :| |
| 47 |
|
| 48 |
It's on the list [1], and I think I've seen 'home' subdirs in the build |
| 49 |
root for a long time. I guess people may be resetting it 'just to be |
| 50 |
sure'. |
| 51 |
|
| 52 |
One thing should be noted though: $HOME may and usually will be |
| 53 |
different than $T. |
| 54 |
|
| 55 |
[1]:http://dev.gentoo.org/~ulm/pms/5/pms.html#x1-11800011.1 |
| 56 |
|
| 57 |
-- |
| 58 |
Best regards, |
| 59 |
Michał Górny |
Archives