Bug 234345

It needs to register them in the openjade catalog.

If openjade is not installed, there is no openjade catalog. Seems silly to
require openjade just to have the catalog in place so that you can register the
dtds in it.

How about openjade %post does that instead ?

I'm not sure if you are still searching space on live cd this way, but maybe
suitable solution could be to use %triggerin -- openjade . Anyway - I see in
spec file "## Clean up pre-docbook-dtds mess caused by broken trigger" - so it
will need some testing before doing that change. 

Changing version to '9' as part of upcoming Fedora 9 GA.
More information and reason for this action is here:
http://fedoraproject.org/wiki/BugZappers/HouseKeeping

*** Bug 449954 has been marked as a duplicate of this bug. ***

To keep discussion here - moved from #449954:
Comment #2 From Jeremy Katz (katzj)  	 on 2008-06-04 11:42 EST  

Requires(post) doesn't help as it still gets pulled in for it.  A trigger might,
but triggers are very tricky to get right.  

A simple fix might be checking if the openjade catalog is there and registering
with it if so, else just leaving it for the openjade %post to later pick up


Comment #3 From Ondrej Vasik (ovasik) 	on 2008-06-05 07:08 EST 	

As I maintain both packages, I will think about that solution - although I would
say trigger in docbook-dtds is slightly better (as the goal is to register
openjade catalogs in docbook-dtds catalogs - and it has almost nothing to do
with openjade package). 

So we have three ways now:
1) Keep it as it is - will save no space on LiveCD
2) triggerin openjade in docbook-dtds - will save space on LiveCD (as openjade
catalogs will be not registered when no openjade available), a bit tricky to
have it running correctly, but it is not affecting other packages
3) move registration to openjade - will save space on LiveCD - but could cause
troubles in some situation (e.g. if you install openjade first, it will not be
registered in docbook-dtds - which could cause some troubles). So registration
has to be doubled (done in both - openjade and docbook-dtds %post - just for
cases that the second package is already installed - that's almost as tricky as
the trigger solution and affects both packages as they have to be synchronized.

I like the trigger solution, as this affects only one package and will have
minimal impact if properly written. But as you said, it could easily be broken
and has to be tested before applying to Fedora (definitely not to F9 or lower to
not break stable release).

We should really get this fixed for F10. This is forcing openjade on the the live cd for no good reason, where we have no space at all for this kind of dead baggage.

Please ?

Ok, I'll try to do it by trigger ... there is already request to remove perl dependency because of FedoraMini, so hopefully we could get rid off both space-eating dependencies.

Ok, done by triggers and built as docbook-dtds-1.0-41.fc10, hopefully it will work as expected and will not break (m)any thing(s) ...Checked with/without installed openjade, removed openjade, updated openjade... so I guess it will be ok... Closing RAWHIDE.

Cool, thanks

This solution didn't actually work, see bug #486257