Bug 1151652
| Summary: | Merge emacs-el into emacs-common | ||||||
|---|---|---|---|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Glenn Morris <rgm+rh> | ||||
| Component: | emacs | Assignee: | Petr Hracek <phracek> | ||||
| Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> | ||||
| Severity: | low | Docs Contact: | |||||
| Priority: | unspecified | ||||||
| Version: | rawhide | CC: | jchaloup, jonathan.underwood, phracek, sergiodj | ||||
| Target Milestone: | --- | ||||||
| Target Release: | --- | ||||||
| Hardware: | Unspecified | ||||||
| OS: | Unspecified | ||||||
| Whiteboard: | |||||||
| Fixed In Version: | emacs-24.4-2.fc21 | Doc Type: | Bug Fix | ||||
| Doc Text: | Story Points: | --- | |||||
| Clone Of: | Environment: | ||||||
| Last Closed: | 2014-10-24 09:53:58 UTC | Type: | Bug | ||||
| Regression: | --- | Mount Type: | --- | ||||
| Documentation: | --- | CRM: | |||||
| Verified Versions: | Category: | --- | |||||
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||
| Cloudforms Team: | --- | Target Upstream Version: | |||||
| Embargoed: | |||||||
| Attachments: |
|
||||||
|
Description
Glenn Morris
2014-10-10 23:43:23 UTC
Created attachment 948795 [details]
Patch to merge the packages
I agree with Glenn's rationale. Here is a patch to merge both packages.
Hi Glen, Sergio, have you discussed this with upstream? Sorry, this is Fedora issue :) (In reply to Sergio Durigan Junior from comment #1) > Created attachment 948795 [details] > Patch to merge the packages > > I agree with Glenn's rationale. Here is a patch to merge both packages. Thanks for the patch. What I missed in your patch is directive like Obsoletes emacs-el which doesn't exist anymore after this update. I am gonna to correct it. You are right, Peter. I thought about it yesterday night, but forgot to resubmit a new version. Anyway, thanks for that. What about to have emacs-el package which will be required by emacs-common?
In the rawhide, can be specified SPEC tag like Recommends which means
that you can uninstall package without troubles if you wants.
From my point of view I would like to have those files separate in packages.
Patch:
Requires(posttrans): %{_sbindir}/alternatives
Requires(post): /sbin/install-info
Requires: %{name}-filesystem
+Requires: %{name}-el
%description common
Emacs is a powerful, customizable, self-documenting, modeless text
Why do you want to have them in separate packages? And with regards to it being uninstallable, I would again like to say that removing the .el files is not a mode of installation that upstream Emacs expects or supports. (I am speaking as an Emacs developer.) What would the point of uninstalling it be? To save 14MB? If you were packaging Emacs today from scratch, I don't think you would come up with this split; so why not simplify future maintenance? Or maybe I misunderstood what you meant by
> In the rawhide, can be specified SPEC tag like Recommends which means
> that you can uninstall package without troubles if you wants.
Hi, well, I would like to explain the situation a bit else. In rawhide we will have a emacs-common and emacs-el packages. And emacs-common files will require emacs-el packages. But as you mentioned el files are part of installation and I will do that as proposed Sergio. The bug is fixed by commit (https://lists.fedoraproject.org/pipermail/scm-commits/Week-of-Mon-20141020/1416577.html) emacs-24.3-28.fc21 has been submitted as an update for Fedora 21. https://admin.fedoraproject.org/updates/emacs-24.3-28.fc21 emacs-24.4-1.fc21 has been submitted as an update for Fedora 21. https://admin.fedoraproject.org/updates/emacs-24.4-1.fc21 emacs-24.4-2.fc21 has been submitted as an update for Fedora 21. https://admin.fedoraproject.org/updates/emacs-24.4-2.fc21 emacs-24.4-2.fc21 has been pushed to the Fedora 21 stable repository. If problems still persist, please make note of it in this bug report. |