Opened 11 years ago
Closed 11 years ago
#503 closed defect (fixed)
Resurrect pdf_builtin
Reported by: | Juergen Reuter | Owned by: | kilian |
---|---|---|---|
Priority: | P0 | Milestone: | v2.2.0 |
Component: | core | Version: | 2.1.1 |
Severity: | normal | Keywords: | |
Cc: |
Description
It's not working. Seems to be the setup of the paths, which traces down through integrations to processes.
Change History (7)
comment:1 Changed 11 years ago by
comment:2 Changed 11 years ago by
Priority: | P3 → P1 |
---|
comment:3 Changed 11 years ago by
One more thing connected to comment1: At the moment, the unit test is copying the PDF table over from its place in share/pdf_builtin or so. But here also the proper test path should be used! This is connected to the qcd_parameters ticket (#513).
comment:4 Changed 11 years ago by
Priority: | P1 → P0 |
---|
The previous (2.1) pdf_builtin SINDARIN file is still not running through as still it is not possible to set other PDFs. But this might be because they are set as optional arguments? (cf. #516). Ranking it up anyhow....
comment:5 Changed 11 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
Done in r4651, including alpha_s access. The PDF set can be defined globally. Local options are indeed not implemented yet, but this is a different ticket (see above). Closing this one.
comment:6 Changed 11 years ago by
Resolution: | fixed |
---|---|
Status: | closed → reopened |
This seems not to be correct in tests/Makefile.am, WHIZARD should know the correct paths (now):
PDF_FILE = cteq6l.tbl $(PDF_FILE): $(top_builddir)/share/pdf_builtin/$(PDF_FILE) cp $< $@ sf_pdf_builtin.run: $(PDF_FILE)
So I guess this has to be removed. Right?
comment:7 Changed 11 years ago by
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
Yes, this was obsolete. Removed in r4668.
With r4551, the path is correctly set again for the installed version. To prevent the dispatch unit tests to depend on external path variables, there a dummy path "." is used. Still, the PDF set cannot be correctly set yet.