From: Pacho Ramos <pacho@gentoo.org>
To: "Michał Górny" <mgorny@gentoo.org>
Cc: gentoo-dev@lists.gentoo.org, dotnet@gentoo.org
Subject: Re: [gentoo-dev] mono-env.eclass: new eclass to be used by most of dotnet packages
Date: Sun, 14 Apr 2013 12:25:27 +0200 [thread overview]
Message-ID: <1365935127.6940.17.camel@localhost> (raw)
In-Reply-To: <20130414120842.46d1a18d@pomiocik.lan>
El dom, 14-04-2013 a las 12:08 +0200, Michał Górny escribió:
> On Sun, 14 Apr 2013 11:59:14 +0200
> Pacho Ramos <pacho@gentoo.org> wrote:
>
> > El dom, 14-04-2013 a las 11:45 +0200, Michał Górny escribió:
> > > On Sun, 14 Apr 2013 11:40:03 +0200
> > > Pacho Ramos <pacho@gentoo.org> wrote:
> > >
> > > > # >=mono-0.92 versions using mcs -pkg:foo-sharp require shared memory, so we set the
> > > > # shared dir to ${T} so that ${T}/.wapi can be used during the install process.
> > > > export MONO_SHARED_DIR="${T}"
> > >
> > > Don't use ${T} in global scope. And just don't export them
> > > in the global scope either.
> >
> > Why not?
>
> Let's start with the fact that ${T} is only partially persistent
> by the words of PMS. I don't know if it's really relevant here but
> you're exporting persistent variables with value based on
> an non-persistent one.
>
> Thinking about it more, it probably would work. As long as you don't
> assume anything about those directories on pkg_*rm() where ${T} would
> have changed already and your variables wouldn't.
>
Yes, they will be needed at compile time, that would explain why no
problem raised for now :/ Thanks for the info
> Also, why are you exporting HOME? PMS does that already...
>
Probably because it's inherited from current mono.eclass, but, are you
sure PMS does that already? There are more examples in the tree (in
eclasses and ebuilds) exporting HOME in similar way (vim.eclass for
example) :/ Or maybe it was started to be exported more recently and
this is only a relic :|
next prev parent reply other threads:[~2013-04-14 10:25 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-04-14 9:40 [gentoo-dev] mono-env.eclass: new eclass to be used by most of dotnet packages Pacho Ramos
2013-04-14 9:45 ` Michał Górny
2013-04-14 9:59 ` Pacho Ramos
2013-04-14 10:08 ` Michał Górny
2013-04-14 10:25 ` Pacho Ramos [this message]
2013-04-14 10:40 ` Michał Górny
2013-04-14 21:02 ` Zac Medico
2013-04-14 10:00 ` Pacho Ramos
2013-05-19 10:31 ` Pacho Ramos
2013-05-24 19:18 ` Pacho Ramos
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1365935127.6940.17.camel@localhost \
--to=pacho@gentoo.org \
--cc=dotnet@gentoo.org \
--cc=gentoo-dev@lists.gentoo.org \
--cc=mgorny@gentoo.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox