Bug 234345 - should not require openjade
Summary: should not require openjade
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: docbook-dtds
Version: 9
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Ondrej Vasik
QA Contact:
URL:
Whiteboard:
: 449954 (view as bug list)
Depends On:
Blocks: FC7Target
TreeView+ depends on / blocked
 
Reported: 2007-03-28 16:27 UTC by Matthias Clasen
Modified: 2009-02-19 04:04 UTC (History)
2 users (show)

Fixed In Version: docbook-dtds-1.0-41.fc10
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2008-09-25 13:20:59 UTC


Attachments (Terms of Use)

Description Matthias Clasen 2007-03-28 16:27:36 UTC
While looking for space to save on the live cd, I noticed that
docbook-dtds requires openjade, which in turn pulls in opensp.

Is that really necessary ? I mean docbook-dtds is just a bunch of
data files, really...

Comment 1 Tim Waugh 2007-03-29 09:29:12 UTC
It needs to register them in the openjade catalog.

Comment 2 Matthias Clasen 2007-03-29 13:52:02 UTC
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 ?

Comment 3 Ondrej Vasik 2008-02-08 23:46:35 UTC
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. 

Comment 4 Bug Zapper 2008-05-14 02:42:45 UTC
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

Comment 5 Ondrej Vasik 2008-06-04 13:56:35 UTC
*** Bug 449954 has been marked as a duplicate of this bug. ***

Comment 6 Ondrej Vasik 2008-06-05 11:10:14 UTC
To keep discussion here - moved from #449954:
Comment #2 From Jeremy Katz (katzj@redhat.com)  	 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@redhat.com) 	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).

Comment 7 Matthias Clasen 2008-09-24 17:33:54 UTC
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 ?

Comment 8 Ondrej Vasik 2008-09-24 18:42:34 UTC
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.

Comment 9 Ondrej Vasik 2008-09-25 13:20:59 UTC
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.

Comment 10 Matthias Clasen 2008-09-25 14:59:20 UTC
Cool, thanks

Comment 11 Tom Lane 2009-02-19 04:04:51 UTC
This solution didn't actually work, see bug #486257


Note You need to log in before you can comment on or make changes to this bug.