Opened 13 years ago
Closed 13 years ago
#31676 closed enhancement (fixed)
openjade @1.3.2 use root catalog in etc/sgml/catalog for SGML processing
Reported by: | fgp (Florian G. Pflug) | Owned by: | drkp (Dan Ports) |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | |
Keywords: | haspatch | Cc: | |
Port: | openjade |
Description (last modified by mf2k (Frank Schima))
openjade by default uses the catalog in share/sgml/catalog for SGML processing. Unlike the XML DTDs, however, none of the SGML DTD ports (like docbook-dsssl) have so far bothered to add themselves to that catalog. I've submitted patches for iso8879 (ticket:31674) and docbook-dsssl (ticket:31656) which change that.
However, in these patches I made the ports register themselves not with the catalog in share/sgml/catalog, but rather with the catalog in etc/sgml/catalog. This made more sense to me, since
- it mimick the way things are done for XML DTDs
- it's the default catalog for xmlcatmgr (which, despite it's name, has a perfectly functional SGML mode)
- it's what e.g. debian does (and probably some other distros too)
So, to make these patches usefull, openjade needs to take the root catalog in etc/sgml/catalog into account. The attached patch makes that happen by adding a reference to etc/sgml/catalog to share/sgml/catalog. Since openjade requires the root catalog to contain a reference to it's own style sheet DTSs (in share/sgml/openjade), the patch also registers share/sgml/openjade/catalog with etc/sgml/catalog.
With my patch applied, things are thus as follows
- etc/sgml/catalog is the root SGML DTD catalog. All packages which contain SGML DTDs should register their package catalogs with that root catalog.
- The above is true also for the style sheet DTDs shipped with openjade, i.e. they are registered with etc/sgml/catalog.
- openjade defaults toshare/sgml/catalog, which contains a reference to etc/sgml/catalog (added by a post-activation hook).
Attachments (1)
Change History (4)
Changed 13 years ago by fgp (Florian G. Pflug)
Attachment: | macports.openjade.diff added |
---|
comment:1 Changed 13 years ago by mf2k (Frank Schima)
Description: | modified (diff) |
---|---|
Keywords: | haspatch added |
Version: | 2.0.3 |
comment:2 Changed 13 years ago by drkp (Dan Ports)
Owner: | changed from macports-tickets@… to dports@… |
---|---|
Status: | new → assigned |
comment:3 Changed 13 years ago by drkp (Dan Ports)
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Thanks, r86411.
Portfile patch. Seem comments for details.