Cannot handle long filenames as input

[GoogleCodeExporter]

Hi,

Seems like the makefile can't handle long filenames. I includ an epslatex 
graphics in my source file as following :

\input{./results/specfp_smt_disp_iss_prof_fifo_4_uops_ready_disp_issued_less_ctr
.tex}

and the error I get is 
make: *** No rule to make target 
`results/specfp_smt_disp_iss_prof_fifo_4_uops_ready_disp_issued_less_ctr\', 
needed by `ooo_fifo_main.d'.

make: Failed to remake makefile `ooo_fifo_main.d'.
make: *** No rule to make target 
`results/specfp_smt_disp_iss_prof_fifo_4_uops_ready_disp_issued_less_ctr .pdf', 
needed by `ooo_fifo_main.pdf'.

I have verified this by shortening the filename. Also by running pdflatex 
directly in the shell it works fine.

thanx.

Original issue reported on code.google.com by abhishek...@gmail.com on 21 Jan 2011 at 5:52

[GoogleCodeExporter]

Following is how the log file looks like
.SECONDEXPANSION:
-include 
./results/specfp_smt_disp_iss_prof_fifo_4_uops_ready_disp_issued_less_ctr\\\
.pdf.gpi.d
ooo_fifo_main.d: $$(call 
graphics-source,./results/specfp_smt_disp_iss_prof_fifo_4_uops_ready_disp_issued
_less_ctr\\\
.pdf)
ooo_fifo_main.pdf ooo_fifo_main._graphics: $$(call 
graphics-target,./results/specfp_smt_disp_iss_prof_fifo_4_uops_ready_disp_issued
_less_ctr\\\
.pdf)
-include 
./results/specint_smt_disp_iss_prof_fifo_4_uops_ready_disp_issued_less_ct\\\
r.pdf.gpi.d
ooo_fifo_main.d: $$(call 
graphics-source,./results/specint_smt_disp_iss_prof_fifo_4_uops_ready_disp_issue
d_less_ct\\\
r.pdf)
ooo_fifo_main.pdf ooo_fifo_main._graphics: $$(call 
graphics-target,./results/specint_smt_disp_iss_prof_fifo_4_uops_ready_disp_issue
d_less_ct\\\
r.pdf)

Original comment by abhishek...@gmail.com on 21 Jan 2011 at 6:09

[GoogleCodeExporter]

Quick note: the makefile doesn't fully support subdirectories, and it's looking 
like it never will.  This will probably work most of the time, but things like 
"make clean" will not work properly with graphics in subdirectories.

That said, this appears to be a real bug and I'm looking at it right now.  If 
you could give me more details, like attaching your log files, that might help 
me find it.

Original comment by shiblon on 22 Jan 2011 at 5:45

• Changed state: Started

[GoogleCodeExporter]

I believe that this can be fixed with a one-line patch.  If you are building 
the makefile from the source, then the attached patch will do it for you (I 
think).  If you are just using the downloaded makefile, look for the line that 
says

-e '  s/.*File `\([^'"'"']*\)'"'"' not found.*/\1/' \        

And add the following directly below it (the backslash needs to be the last 
thing on the line):

-e '  s/[[:cntrl:]]//' \

IF that doesn't fix it, then we have another one lurking somewhere, but it 
definitely fixed the error for me.

Original comment by shiblon on 22 Jan 2011 at 5:59

[GoogleCodeExporter]

Actually attaching patch.  Also, this is submitted, and is available as 
rb199832dda5e.

Original comment by shiblon on 22 Jan 2011 at 6:05

Attachments:

• missing-inputs.patch

[GoogleCodeExporter]

Ok I have just moved to 2.2.0-rc8 and the problem seems to have resolved. Even 
without the one line patch. Earlier I was using 2.2.0-rc6.

Thanx for a quick reply.

Original comment by abhishek...@gmail.com on 22 Jan 2011 at 12:43

[GoogleCodeExporter]

Hey, that's great.  There was still a bug, though, so I'm glad you submitted 
the issue report.

Original comment by shiblon on 22 Jan 2011 at 5:15

• Changed state: Verified

[GoogleCodeExporter]

so is the one line patch needed anymore? 

thanx for your makefile it certainly is convenient to automate things.

Original comment by abhishek...@gmail.com on 22 Jan 2011 at 6:31

[GoogleCodeExporter]

The one-line patch is now part of the main distribution: 2.2.0-rc9

Enjoy!

And you're welcome.  :-)

Original comment by shiblon on 22 Jan 2011 at 7:11