johnhaley81 / gyp

Automatically exported from code.google.com/p/gyp
0 stars 0 forks source link

gyp analyzer doesn't match targets which define inputs with '.' relative path values. #487

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?

In chromium code:
1. Create a gyp target which includes "build/java.gypi" (let call it 'Test1'), 
add 'Test1' to 'all' target.
2. Define 'java_in_dir' variable with a value like './'
3. Prepare in.json for analyzer with any java file which would be dependent to 
Test1, something like 'path_to_our_gyp/src/org/some_path/some_file.java' in 
'files' and 'Test1' in 'targets' (Don't forget to create this file).
3. Run build/gyp_chromium --analyzer in.json out.json

More simple scenario: just create a gyp target with an action that has 
'./some_file' in its 'inputs'

What is the expected output? 

analyzer must print something like:

Found dependency
Supplied targets that depend on changed files:
    Test1
Targets that require a build:
    Test1

What do you see instead?

It will print instead:

No dependencies
Supplied targets that depend on changed files:
Targets that require a build:

Original issue reported on code.google.com by kind...@yandex-team.ru on 15 Apr 2015 at 11:54

GoogleCodeExporter commented 9 years ago
I'm already making a CL to fix this.

Original comment by kind...@yandex-team.ru on 15 Apr 2015 at 11:56

GoogleCodeExporter commented 9 years ago
But I can't make a review. It always says: Access to 
https://codereview.chromium.org is denied (server returned HTTP 401)

this is my patch attached

Original comment by kind...@yandex-team.ru on 15 Apr 2015 at 1:11

Attachments: