| Summary: | Add `emacs' use flag to dev-util/bzr-0.6.2 ebuild | ||
|---|---|---|---|
| Product: | Gentoo Linux | Reporter: | Christian Schlotter <again> |
| Component: | New packages | Assignee: | Anders Rune Jensen (RETIRED) <arj> |
| Status: | RESOLVED FIXED | ||
| Severity: | enhancement | CC: | marienz |
| Priority: | High | ||
| Version: | unspecified | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Package list: | Runtime testing required: | --- | |
| Attachments: |
bzr-0.6.2.ebuild
files/70bzr-gentoo.el diff to previous ebuild files/70bzr-gentoo.el bzr-0.6.2.ebuild files/70bzr-gentoo.el |
||
|
Description
Christian Schlotter
2006-01-26 04:28:13 UTC
Created attachment 78149 [details]
bzr-0.6.2.ebuild
Created attachment 78150 [details]
files/70bzr-gentoo.el
Created attachment 78151 [details, diff]
diff to previous ebuild
Created attachment 78152 [details]
files/70bzr-gentoo.el
Removed unnecessary line.
Why are you using "(load "bzr-mode.el" nil t t)"? If I read the documentation right the final "t" will make it ignore bzr-mode.elc which seems a waste after creating and installing it :) and I do not see a good reason for hiding the load message either. How about simply (load "bzr-mode")? Created attachment 78274 [details]
bzr-0.6.2.ebuild
Spotted a typo.
Created attachment 78276 [details] files/70bzr-gentoo.el (In reply to comment #5) > Why are you using "(load "bzr-mode.el" nil t t)"? If I read the documentation > right the final "t" will make it ignore bzr-mode.elc which seems a waste after > creating and installing it :) and I do not see a good reason for hiding the > load message either. How about simply (load "bzr-mode")? Marien, you are right. Next time I will look at the documentation instead of just copying a line without checking it. Thanks! Added this as part of other changes in bug 120487. Thanks for reporting! |