Re: using variant_dir but strip out the variant_dir from source files

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 <[email protected]> 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 <[email protected]> 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 <[email protected]> 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
[email protected]
https://pairlist4.pair.net/mailman/listinfo/scons-users