#50220 closed defect (fixed)
p5-sgmlspm: installs sgmlspl.pl[-5.22] instead of sgmlspl[-5.22]
| Reported by: | mojca (Mojca Miklavec) | Owned by: | lockhart (Thomas Lockhart) |
|---|---|---|---|
| Priority: | Normal | Milestone: | |
| Component: | ports | Version: | |
| Keywords: | Cc: | dbevans (David B. Evans) | |
| Port: | p5-sgmlspm |
Description
David, I believe that the change in r144025 broke the port a bit. It previously installed:
/opt/local/bin/sgmlspl-5.22 /opt/local/libexec/perl5.22/sgmlspl
while now it installs
/opt/local/bin/sgmlspl.pl-5.22 /opt/local/libexec/perl5.22/sgmlspl.pl
The whole documentation folder is also missing now.
See #50219.
Attachments (1)
Change History (8)
Changed 9 years ago by mojca (Mojca Miklavec)
| Attachment: | sgmlspm-remove-pl.diff added |
|---|
comment:1 follow-up: 2 Changed 9 years ago by mojca (Mojca Miklavec)
I attached a relatively ugly patch. Can someone please take a look (and potentially suggest something better), so that we can proceed with docbook-utils?
comment:2 Changed 9 years ago by lockhart (Thomas Lockhart)
Replying to mojca@…:
I attached a relatively ugly patch. Can someone please take a look (and potentially suggest something better), so that we can proceed with docbook-utils?
I'm not sure where these executable renamings came from; afaik there was a time in the past when it was not an issue. Which is why I've been slow to see the problems I guess.
I haven't looked at the commit history but I just reinstalled p5.22-sgmlspm yesterday and did not see this particular issue.
Will look at the overall problem of sgmlspm and its interactions with docbook-utils this afternoon and evening.
comment:3 follow-up: 4 Changed 9 years ago by mojca (Mojca Miklavec)
When did you last run selfupdate and/or svn up? David made the change relatively recently. You can check whether you have version 1.1 installed (port installed p5.22-sgmlspm). Unless there is something weird about my system, the build of 1.1 results in the extra .pl file extension.
The port has been at an old version for a very long time. The update switched to a different build system and that might have caused the problem. (Note that this is still a relatively old release with some big fat red "warning" on CPAN saying "unauthorized".)
comment:4 Changed 9 years ago by lockhart (Thomas Lockhart)
Replying to mojca@…:
When did you last run selfupdate and/or
svn up? David made the change relatively recently. You can check whether you have version 1.1 installed (port installed p5.22-sgmlspm). Unless there is something weird about my system, the build of 1.1 results in the extra .pl file extension.The port has been at an old version for a very long time. The update switched to a different build system and that might have caused the problem. (Note that this is still a relatively old release with some big fat red "warning" on CPAN saying "unauthorized".)
Yeah, I update every week or two from svn. Not sure how it slipped by, but perhaps I didn't have it installed? There are other XML-related ports with outstanding issues that I've been occasionally trying to reproduce by reinstalling packages...
comment:5 Changed 9 years ago by mojca (Mojca Miklavec)
So do you agree with the proposed patch or do you have other suggestions/wishes? I would like to close #48365.
comment:6 Changed 9 years ago by mojca (Mojca Miklavec)
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
comment:7 Changed 9 years ago by lockhart (Thomas Lockhart)
Will open a new issue if additional cleanup is useful.
A patch to remove .pl from sgmlspl