[Bug 478613] Review Request: ledger - A powerful command-line double-entry accounting system

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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=478613





--- Comment #3 from Michel Alexandre Salim <michel.sylvan@xxxxxxxxx>  2009-03-14 23:02:11 EDT ---
(In reply to comment #2)
> (In reply to comment #1)
> > - If packaging Emacs modes, you might want to consider making a separate
> > subpackage that depends on emacs. See for instance how I handle it for vala:
> > http://cvs.fedoraproject.org/viewvc/devel/vala/vala.spec?revision=1.36
> 
> I think this is overkill for small files (like .el files).  Plus, I can't
> picture wanting emacs installed because I installed ledger.

Sure, you don't want to get emacs pulled in by default. Which is why I
suggested splitting off the emacs editing modes; this is the convention for
other packages that come with emacs modes; see e.g.
emacs-{gambit,git,gnuplot,lua,mercurial,pyrex,vala}). All of these are split
off from their main packages, so that the main package itself does not depend
on Emacs.

The other alternatives you could do is
1) Not package the Emacs mode files. That would be a shame
2) Own /usr/share/emacs and /usr/share/emacs/site-packages. This is
problematic. Not owning these mean that if a user install ledger on a system
*without* emacs installed, uninstalling ledger means these directories will be
left dangling, as they are not registered as belonging to the package.

On the other hand, owning these would also be problematic and against the
guidelines (though some packages still do this): as much as possible, a package
should not own directories that belong to other packages, in case the two
specify different permissions for the directories / files concerned. There are
exceptions -- bash-completion support scripts are considered trivial enough
that packages normally just own /etc/bash_completion.d rather than pull in
bash-completion as a dependency.

Hope that clarifies things. Now that the lowest supported Fedora version is
F-9, whose Emacs comes with pkgconfig, properly packaging Emacs modes is not
hard (you don't have to hardcode the Emacs version anymore). Unless you want to
target Red Hat Enterprise Linux (and CentOS).. so I'd probably suggest either
dropping the .el files or packaging them properly.

-- 
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.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]