Split actions/rules with process_outputs_as_sources=1 as _prebuild project for MSVC about gyp

[GoogleCodeExporter]

We don't necessarily need to get rid of process_outputs_as_sources.  The MSVS 
generator could automatically generate an auxiliary dependency target for rules 
and 
actions that runs the scripts, and the outputs (when process_outputs_as_sources 
is 
true) could be compiled in the main target.

Original comment by [email protected] on 8 Jul 2009 at 3:18

[GoogleCodeExporter]

You are right.

So my proposal (since I can't rename the title)

- For every action and rules that have 'process_outputs_as_sources': 1,
- On MSVC only,
- Generate a '$(project)_prebuild.vcproj' that contains those steps, depends on 
the same 
dependencies than $(project).
- Have $(project)dependends on '$(project)_prebuild'.

Original comment by [email protected] on 9 Jul 2009 at 2:23

[GoogleCodeExporter]

Original comment by [email protected] on 9 Jul 2009 at 7:14

• Changed title: Split actions/rules with process_outputs_as_sources=1 as _prebuild project for MSVC

[GoogleCodeExporter]

Original comment by [email protected] on 6 Aug 2009 at 6:33

• Changed state: Accepted

[GoogleCodeExporter]

Original comment by [email protected] on 18 Aug 2009 at 8:06

• Added labels: MSVS

[GoogleCodeExporter]

He accepted but forgot to make himself owner

Original comment by [email protected] on 19 Oct 2009 at 5:38

[GoogleCodeExporter]

No, I didn't forget.  Taking my name back off.

The "process" (if you want to dignify it) that bradnelson and I were following 
was to 
move items to Accepted to indicate it at least got a cursory project-level 
triage 
(bradnelson and I going through the new issues) and that the Priority and 
keywords 
are sane.  A name attached is what does or doesn't indicate a specific 
individual has 
intends to work on it.

That's not set in stone, it's just what we started doing the couple of times we 
went 
down the list.

Original comment by [email protected] on 17 Nov 2009 at 1:16

[GoogleCodeExporter]

We did this for Xcode in r762. It worked. We ought to fix it for MSVS too.

Original comment by [email protected] on 25 Jun 2010 at 9:01