Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Autotools with SUBDIRS #21

Open
novns opened this issue Apr 15, 2016 · 5 comments · May be fixed by #31
Open

Autotools with SUBDIRS #21

novns opened this issue Apr 15, 2016 · 5 comments · May be fixed by #31

Comments

@novns
Copy link

novns commented Apr 15, 2016

I got the "Matched file does not exist" error when failed source file is located in subdirectory specified in automake's SUBDIRS variable.

Example of output from make:

make  all-recursive
make[1]: Entering directory '/home/novns/tmp/testmake'
Making all in src
make[2]: Entering directory '/home/novns/tmp/testmake/src'
  CC       testmake.o
testmake.c: In function 'main':
testmake.c:3:5: error: unknown type name 'error'
     error is here!
     ^
@novns
Copy link
Author

novns commented Apr 15, 2016

Exact error:
Matched file does not exist: /home/novns/tmp/testmake/testmake.c

@noseglid
Copy link
Member

Hmm not sure how to handle this. I don't think a regex can cope with this issue.

Possible (if merged) noseglid/atom-build#398 can help give the base for solving this. The matching function could do more intelligent parsing (e.g finding the Entering directory... and interpreting subsequent files from that directory).

@oli-obk
Copy link

oli-obk commented May 12, 2016

@novns
Copy link
Author

novns commented Oct 25, 2016

As a temporary solution one can apply the bad and ugly fix to build/lib/error-matcher.js:

let file = 'src/' + match.file;

@AndreasDahl AndreasDahl linked a pull request Dec 22, 2016 that will close this issue
@rpuntaie
Copy link

Could one try the matched relative path with the root of every loaded project (=directory)? So one could add some sub-directories to the project view and then find the matched file.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants