[Bug 483364] Review Request: EekBoek - Bookkeeping software for small and medium-size businesses

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=483364





--- Comment #5 from Mamoru Tasaka <mtasaka at ioa.s.u-tokyo.ac.jp>  2009-03-19 14:36:17 EDT ---
(In reply to comment #4)
> Thanks for your kind and constructive feedback. Most items are trivially fixed,
> which I will do in a next release.
> 
> The requires/provides "%{name}-db" seems to be confusing, which may be an
> indication that the intention is not clear.  Or wrong.
> 
> As you may have guessed, EekBoek supports both SQLite and PostgreSQL as
> database backend (and probably more to come). It is tempting to bundle the
> SQLite backend with the main package (that is what Debian does) but I
> personally do not like having to install a package that one doesn't need.
> 
> What would be a better approach for this? 

I would say that bundling SQLite backend to main package is
undesired solution.

In such case I recommend
- not to use virtual "Provodes/Requires: %{name}-db"
- write some text named "README.Fedora", for example to inform
  that
  - BekBook needs one of the database backends
  - Currently Fedora supports SQLite and PostgreSQL
  - The admin has to choose either of them and install
    the choosed one by "yum install BekBook-db-<choosed backend>"
    by him/herself
  and add to the "%description" of BekBook package some sentence
  like "Please read README.Fedora"

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.