[Scons-users] using variant_dir but strip out the variant_dir from source files

[Bill Deegan]

Did you try to monkey path it in site_scons/site_init.py ?

Do you create new Environment()'s all over the place?
Or do you create a few and then clone them places?
You can set env['_concat']

Hacking default.py is a bad idea in general, and a very bad idea unless
your using the scons-local scons package included with your sources (in
which case it's still a bad idea)

On Wed, Oct 24, 2018 at 1:03 PM Hua Yanghao <huayanghao at gmail.com> wrote:

> correction: my test shows it is the *former*.
> On Wed, Oct 24, 2018 at 10:02 PM Hua Yanghao <huayanghao at gmail.com> wrote:
> >
> > Bill, I tried but enlighten me please.
> >
> > The problem is when scons took the defaults.py settings ... is that
> > before the first line of code in SConstruct is being called or after
> > ... my test shows it is the latter so whatever I change in
> > SConstruct/SConscript does not make the function exported to the
> > environment where the command string is expanded ...
> > On Wed, Oct 24, 2018 at 8:22 PM Bill Deegan <bill at baddogconsulting.com>
> wrote:
> > >
> > > you should not have to change defaults.py (ever.. it's python you can
> reach into anything and change anything in your SConstruct/sconscript/etc)
> > >
> > > So you want the string to be a/b/c.c and not build/a/b/c.c
> > >
> > >
> > > On Thu, Oct 18, 2018 at 9:07 AM Hua Yanghao <huayanghao at gmail.com>
> wrote:
> > >>
> > >> Hi Bill, I am chdir only to the top level folder "build", and want to
> > >> build CC a/b/c.c instead of CC build/a/b/c.c
> > >>
> > >> I got this working but wonder if there is a way that I do not need to
> > >> change the Defaults.py file.
> > >> On Thu, Oct 18, 2018 at 6:04 PM Bill Deegan <
> bill at baddogconsulting.com> wrote:
> > >> >
> > >> > So you have a file:
> > >> > a/b/c.c
> > >> > variant dir to
> > >> > build/a/b/c.c
> > >> >
> > >> > you're chdir'ing into
> > >> > build/a/b ?
> > >> >
> > >> > And you want the command line to be
> > >> > compiler c.c
> > >> >
> > >> > ?
> > >> >
> > >> > On Thu, Oct 18, 2018 at 6:30 AM Hua Yanghao <huayanghao at gmail.com>
> wrote:
> > >> >>
> > >> >> Update: it looks like the _concat()-like stuff can only be put in
> the
> > >> >> SCons/Defaults.py otherwise not all environment will get it.
> > >> >> Not sure if there is a way that I do not need to modify the
> > >> >> Defaults.py but still able to add one function to the default
> > >> >> environment construction.
> > >> >>
> > >> >> Thanks,
> > >> >> Yanghao
> > >> >> On Thu, Oct 18, 2018 at 12:07 AM Hua Yanghao <huayanghao at gmail.com>
> wrote:
> > >> >> >
> > >> >> > Looked into the Action.py a little bit, doesn't seem to really
> have a
> > >> >> > controllable print for chdir messages.
> > >> >> > I tested below patch which works for me:
> > >> >> >
> > >> >> > diff --git i/tools/scons-3.0.1/scons-local-3.0.1/SCons/Action.py
> > >> >> > w/tools/scons-3.0.1/scons-local-3.0.1/SCons/Action.py
> > >> >> > index ce5471d..ee2a5c1 100644
> > >> >> > --- i/tools/scons-3.0.1/scons-local-3.0.1/SCons/Action.py
> > >> >> > +++ w/tools/scons-3.0.1/scons-local-3.0.1/SCons/Action.py
> > >> >> > @@ -647,7 +647,8 @@ class _ActionAction(ActionBase):
> > >> >> >                                 show=_null,
> > >> >> >                                 execute=_null,
> > >> >> >                                 chdir=_null,
> > >> >> > -                               executor=None):
> > >> >> > +                               executor=None,
> > >> >> > +                               chdir_verbose=False):
> > >> >> >          if not is_List(target):
> > >> >> >              target = [target]
> > >> >> >          if not is_List(source):
> > >> >> > @@ -691,7 +692,8 @@ class _ActionAction(ActionBase):
> > >> >> >                  cmd = self.strfunction(target, source, env)
> > >> >> >              if cmd:
> > >> >> >                  if chdir:
> > >> >> > -                    cmd = ('os.chdir(%s)\n' % repr(chdir)) + cmd
> > >> >> > +                    if chdir_verbose:
> > >> >> > +                        cmd = ('os.chdir(%s)\n' % repr(chdir))
> + cmd
> > >> >> >                  try:
> > >> >> >                      get = env.get
> > >> >> >                  except AttributeError:
> > >> >> > @@ -718,7 +720,7 @@ class _ActionAction(ActionBase):
> > >> >> >              finally:
> > >> >> >                  if save_cwd:
> > >> >> >                      os.chdir(save_cwd)
> > >> >> > -        if cmd and save_cwd:
> > >> >> > +        if cmd and save_cwd and chdir_verbose:
> > >> >> >              print_func('os.chdir(%s)' % repr(save_cwd), target,
> source, env)
> > >> >> >
> > >> >> >          return stat
> > >> >> > On Wed, Oct 17, 2018 at 11:54 PM Hua Yanghao <
> huayanghao at gmail.com> wrote:
> > >> >> > >
> > >> >> > > I finally implemented something like _concat() so I can use it
> > >> >> > > directly in the COMSTR definition to strip out the additional
> path.
> > >> >> > > However now scons trying to be very verbose and every single
> > >> >> > > os.chdir() is being printed (chdir="..." used for all
> builders). Is
> > >> >> > > there a way to disable this os.chdir() print message without
> modifying
> > >> >> > > the scons source code? Thanks.
> > >> >> > > On Wed, Oct 17, 2018 at 9:55 PM Hua Yanghao <
> huayanghao at gmail.com> wrote:
> > >> >> > > >
> > >> >> > > > Dear Scons User,
> > >> >> > > > I am having maybe an odd request. The background is there
> are some
> > >> >> > > > compilers out there produces all kinds of output in your
> current
> > >> >> > > > directory which you do not have control. I wanted to combine
> the
> > >> >> > > > variant_dir feature plus the chdir kw arguments to builders,
> to be
> > >> >> > > > able to invoke those compilers directly inside the
> variant_dir. And I
> > >> >> > > > still want to keep the source duplication in the variant_dir
> as it
> > >> >> > > > will be a snapshot of only the used files for a particular
> built out
> > >> >> > > > of thousands of irrelevant files. As a side effect now the
> source
> > >> >> > > > files are all prefixed with the variant_dir which becomes
> "invalid"
> > >> >> > > > because the compiler is already invoked in the variant_dir
> instead of
> > >> >> > > > the top level directory.
> > >> >> > > >
> > >> >> > > > Is there a way (e.g. a special env variable) already that
> when I write
> > >> >> > > > my builders instead of "$COMPILER $SOURCE", I can use
> something like
> > >> >> > > > "$COMPILER $ORIGINAL_SOURCE"? Or do I have to wrap around the
> > >> >> > > > $COMPILER with another command such that I can do things like
> > >> >> > > > "$COMPILER_WRAPPED $VARIANT_DIR $SOURCE"? I know the second
> one
> > >> >> > > > definitely can work but wondering if there are better ways
> already in
> > >> >> > > > scons.
> > >> >> > > >
> > >> >> > > > Best Regards,
> > >> >> > > > Yanghao
> > >> >> _______________________________________________
> > >> >> Scons-users mailing list
> > >> >> Scons-users at scons.org
> > >> >> https://pairlist4.pair.net/mailman/listinfo/scons-users
> > >> >
> > >> > _______________________________________________
> > >> > Scons-users mailing list
> > >> > Scons-users at scons.org
> > >> > https://pairlist4.pair.net/mailman/listinfo/scons-users
> > >> _______________________________________________
> > >> Scons-users mailing list
> > >> Scons-users at scons.org
> > >> https://pairlist4.pair.net/mailman/listinfo/scons-users
> > >
> > > _______________________________________________
> > > Scons-users mailing list
> > > Scons-users at scons.org
> > > https://pairlist4.pair.net/mailman/listinfo/scons-users
> _______________________________________________
> Scons-users mailing list
> Scons-users at scons.org
> https://pairlist4.pair.net/mailman/listinfo/scons-users
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://pairlist4.pair.net/pipermail/scons-users/attachments/20181024/6bcac025/attachment.html>