#20806 closed defect (fixed)
yelp should not depend on lzmautils
Reported by: | vinc17@… | Owned by: | dbevans (David B. Evans) |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | 1.7.1 |
Keywords: | Cc: | ||
Port: | yelp |
Description
The dependency on lzmautils doesn't seem to be necessary (I suppose that the library can be used to open lzma-compressed files) and prevents yelp from being built when the xz-devel port has been installed because it conflicts with the lzmautils port (lzmautils has been superseded by xz-utils). This dependency should be removed (or a lzma variant could be added, but lzma-compressed files should no longer be used anyway).
Change History (4)
comment:1 Changed 15 years ago by dbevans (David B. Evans)
Status: | new → assigned |
---|
comment:2 Changed 15 years ago by dbevans (David B. Evans)
comment:3 Changed 15 years ago by dbevans (David B. Evans)
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Dependency removed in r56250 with comment as to the reason for the future.
Note however, that libarchive also depends on lzmautils and is used by gvfs to implement its archive backend.
Since the xz-devel port does not provide liblzmadec only the conflicting utilities, perhaps another solution would be to make a liblzmadec port from the same source as lzmautils that only installs the library and include files and use that as a dependency for libarchive and yelp.
comment:4 Changed 15 years ago by vinc17@…
libarchive no longer depends on lzmautils, and doesn't need the obsolete liblzmadec library either, as xz-utils supports both the old lzma format and the new format via its liblzma library. If libarchive can't use it, it is broken.
After looking a bit, yelp doesn't compress anything with lzma but can transparently convert files of mime-type application/x-lzma that are lzma compressed info and man files to the yelp html format. A search of /opt/local doesn't show any (and I have the full gnome set of packages installed) so it looks like it is safe to remove.
Note however that this in one of those instances of opportunistic linking -- yelp will use liblzmadec if it finds it and will disable lzma support if it doesn't but has no configure option to unconditionally turn it off.
The usual practice in this case has been to include the dependency to ensure that everyone's build works the same.