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)

macports.openjade.diff (3.0 KB) - added by fgp (Florian G. Pflug) 13 years ago.
Portfile patch. Seem comments for details.

Download all attachments as: .zip

Change History (4)

Changed 13 years ago by fgp (Florian G. Pflug)

Attachment: macports.openjade.diff added

Portfile patch. Seem comments for details.

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: newassigned

comment:3 Changed 13 years ago by drkp (Dan Ports)

Resolution: fixed
Status: assignedclosed

Thanks, r86411.

Note: See TracTickets for help on using tickets.