Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

#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)

sgmlspm-remove-pl.diff (1.4 KB) - added by mojca (Mojca Miklavec) 3 years ago.
A patch to remove .pl from sgmlspl

Download all attachments as: .zip

Change History (8)

Changed 3 years ago by mojca (Mojca Miklavec)

Attachment: sgmlspm-remove-pl.diff added

A patch to remove .pl from sgmlspl

comment:1 Changed 3 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 in reply to:  1 Changed 3 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 Changed 3 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 in reply to:  3 Changed 3 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 3 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 3 years ago by mojca (Mojca Miklavec)

Resolution: fixed
Status: newclosed

comment:7 Changed 3 years ago by lockhart (Thomas Lockhart)

Will open a new issue if additional cleanup is useful.

Note: See TracTickets for help on using tickets.