Opened 9 years ago

Closed 9 years ago

#5151 closed bug (fixed)

OptionalPackage: Bzip: contains broken symlinks

Reported by: mmadia Owned by: scottmc
Priority: normal Milestone: R1
Component: - General Version: R1/Development
Keywords: Cc:
Blocked By: Blocking:
Has a Patch: no Platform: All

Description

hrev34679. The symbolic links that are contained within the package point to /boot/bz2/boot/common/bin/<file>. Affected files: bzcmp, bzegrep, bzfgrep, & bzless.

Change History (5)

comment:1 Changed 9 years ago by augiedoggie

There are other minor issues with it as well. /boot/common/lib/libbz2.so.1.0 is a real file instead of being a symlink to /boot/common/lib/libbz2.so.1.0.4

In addition, although it's not part of the bzip2 build scripts, most operating systems also provide a libbz2.so symlink which points at libbz2.so.1.0.4

I've also noticed that some packagers fix the library versions to be 1.0.5 to match the package version. I think this one is optional though as Ubuntu seems to keep libbz2.so.1.0.4 even though I have 1.0.5 installed.

comment:2 Changed 9 years ago by scottmc

These issues can probably be fixed right up with some minor changes to the .bep file for bzip. This might have been casued by issues with haikuporter. It might be that it was created with an early rev when the distro option was added but not fully debugged. Maybe it's still broken? I might be able to take a closer look at it soon, now that finals are over.

comment:3 Changed 9 years ago by augiedoggie

Not to drag this ticket off topic, but...

I don't think the 'distro' technique used by haikuporter is ever going to work for all packages. It is currently substituting 'make install' for 'make install DESTDIR=/path'. Not all packages use make for the build system, of those, not all are going to respect the DESTDIR variable, and some packages require some fine tuning after the 'make install' portion is completed(graphviz comes to mind). I think it would be too much work to change every package to work like this.

The real solution would be to port something like checkinstall/installwatch or a similar utility and incorporate that into the haikuporter script. I had a go at porting installwatch several months ago, but, it didn't get along nicely with the Haiku runtime_loader. :(

comment:4 Changed 9 years ago by scottmc

Fixed in hrev34781. The patch on HaikuPorts for bzip2 has been updated to created the proper links now.

comment:5 Changed 9 years ago by scottmc

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.