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

[Hua Yanghao]

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