From: Anton Starikov <antst@ifm.liu.se>
To: gentoo-dev@lists.gentoo.org
Subject: Re: [gentoo-dev] per-package environment variables.
Date: Thu, 02 Sep 2004 00:39:13 +0200 [thread overview]
Message-ID: <41364F91.7090002@ifm.liu.se> (raw)
In-Reply-To: <200409011556.12085.ribosome@gentoo.org>
Olivier Fisette wrote:
>>Anyway, if we going to include both, files and inlined vars, we have to
>>have a bit complicated parser of lines. In such case we even don't need
>>[], we can leave as
>>=app-sci/pymol-0.95* gcc_low_opt CC="gcc"
>>"=" specify enough that we have inlined expression.
>
>
> I don't think so. What if you want to source a bash script containing
> functions to manipulate variables, such as filtering or replacing CFLAGS?
> There is no "=" in such a command. If inline is used, we'll need delimiters.
>
> However, I don't think inline definitions are that important. A first
> implementation dealing only with "atom [env_file] [...]" in package.env would
> be enough, certainly.
>
Even more, I can say that my first idea was idea with delimiters or some
kind of brackets. But after time I've realized that it's looks a bit
ugly and not intuitive. Moreover, I really believe that if you want to
include something really complicated, better to do it via separate file,
rather via inlining, by this you will decrease probability that file
will be parsed wrong. Moreover, I also suggest that inline definitions
not such important, more, I believe that in 99.9% cases you will want to
inline something like VAR="foo bar". So, I think it not worst to make
limitation about inlining, only simple variable definitions. Rest goes
to separate files. In such case we have somehow easy and intuitive (I
hope) syntax, which not looks ugly (at least for me :)), simple parsing
and so on.
But of course, implementing of "atom env_file1 env_file2 ..." is
simplest for now, at least from my ponit of view. (I'm really not great
specialist in python, and really didn't pass throw all portage scripts,
but I've found that I can really implement easy this syntax for adding
about 5-10 lines of code into portage, via using methods that already
exist there)
--
Anton Starikov
--
gentoo-dev@gentoo.org mailing list
next prev parent reply other threads:[~2004-09-01 22:39 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-08-31 22:26 [gentoo-dev] per-package environment variables Antst GD
2004-08-31 22:34 ` Mike Frysinger
2004-08-31 22:44 ` Antst GD
2004-09-08 18:19 ` Ned Ludd
2004-08-31 23:28 ` Mike Frysinger
2004-09-01 0:00 ` Anton Starikov
2004-09-01 0:13 ` Anton Starikov
2004-09-01 3:05 ` Olivier Fisette
2004-09-01 11:08 ` Anton Starikov
2004-09-01 15:53 ` Robin H. Johnson
2004-09-01 19:56 ` Olivier Fisette
2004-09-01 22:39 ` Anton Starikov [this message]
2004-09-02 22:49 ` Anton Starikov
2004-09-02 23:30 ` Olivier Fisette
2004-09-03 1:36 ` Anton Starikov
2004-09-03 1:35 ` Robin H. Johnson
2004-09-03 2:05 ` Olivier Fisette
2004-09-03 2:53 ` Anton Starikov
2004-09-01 0:26 ` Nicholas Jones
2004-09-01 0:51 ` Anton Starikov
2004-09-01 14:02 ` aye
2004-09-01 14:22 ` Anton Starikov
2004-09-01 15:59 ` Robin H. Johnson
2004-09-01 18:42 ` aye
2004-09-01 18:46 ` Robin H. Johnson
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=41364F91.7090002@ifm.liu.se \
--to=antst@ifm.liu.se \
--cc=gentoo-dev@lists.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