public inbox for gentoo-dev@lists.gentoo.org
 help / color / mirror / Atom feed
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


  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