Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 80418

Summary: new ebuild: app-emacs/remember
Product: Gentoo Linux Reporter: Matthew Kennedy (RETIRED) <mkennedy>
Component: New packagesAssignee: Josh Glover (RETIRED) <jmglov>
Status: RESOLVED FIXED    
Severity: normal CC: egor.pervuninski, evan.monroig
Priority: High    
Version: unspecified   
Hardware: All   
OS: All   
Whiteboard:
Package list:
Runtime testing required: ---
Attachments: ebuild for app-emacs/remember version 1.9
metadata for app-emacs/remember
site file for app-emacs/remember
remember-1.9.ebuild; removed explicit setting of ${S}
New version of remember-1.9.ebuild; added local USE flags for optional dependencies
Patch to use.local.desc containing new local USE flags

Description Matthew Kennedy (RETIRED) gentoo-dev 2005-02-01 21:33:14 UTC
see attachments http://bugs.gentoo.org/show_bug.cgi?id=27752
Comment 1 Mamoru KOMACHI (RETIRED) gentoo-dev 2005-05-03 02:06:33 UTC
*** Bug 90504 has been marked as a duplicate of this bug. ***
Comment 2 Evan Monroig 2006-08-08 17:38:28 UTC
I created an ebuild for app-emacs/remember for the last release 1.9 (of a few days ago) based on the ebuild of app-emacs/planner.
Comment 3 Evan Monroig 2006-08-08 17:39:35 UTC
Created attachment 93807 [details]
ebuild for app-emacs/remember version 1.9
Comment 4 Evan Monroig 2006-08-08 17:40:09 UTC
Created attachment 93808 [details]
metadata for app-emacs/remember
Comment 5 Evan Monroig 2006-08-08 17:40:41 UTC
Created attachment 93809 [details]
site file for app-emacs/remember
Comment 6 Josh Glover (RETIRED) gentoo-dev 2006-08-11 23:35:14 UTC
Created attachment 94032 [details]
remember-1.9.ebuild; removed explicit setting of ${S}

Evan, pretty good ebuild. I uploaded a new one with just one difference:

--- remember-1.9.ebuild 2006-08-12 15:28:32.060216571 +0900
+++ remember-1.9.ebuild.jmglov  2006-08-12 15:28:15.581898609 +0900
@@ -20,8 +20,6 @@

 SITEFILE=50remember-gentoo.el

-S="${WORKDIR}/${P}"
-
 src_compile() {
        emake || die
 }

No need to set S manually to the default. :)

Now, here is what I get when I emerge it:

>>> Compiling source in /var/tmp/portage/remember-1.9/work/remember-1.9 ...
Wrote /var/tmp/portage/remember-1.9/work/remember-1.9/read-file-name.elc
Done
While compiling toplevel forms in file /var/tmp/portage/remember-1.9/work/remember-1.9/remember-bibl.el:
  !! File error (("Cannot open load file" "url"))
Done
While compiling toplevel forms in file /var/tmp/portage/remember-1.9/work/remember-1.9/remember-bbdb.el:
  !! File error (("Cannot open load file" "bbdb-com"))
Done
Wrote /var/tmp/portage/remember-1.9/work/remember-1.9/remember-diary.elc
Done
While compiling toplevel forms in file /var/tmp/portage/remember-1.9/work/remember-1.9/remember-blosxom.el:
  !! File error (("Cannot open load file" "emacs-wiki-blosxom"))
Done
Wrote /var/tmp/portage/remember-1.9/work/remember-1.9/remember.elc
Done
While compiling toplevel forms in file /var/tmp/portage/remember-1.9/work/remember-1.9/remember-emacs-wiki-journal.el:
  !! File error (("Cannot open load file" "emacs-wiki-journal"))
Done
While compiling toplevel forms in file /var/tmp/portage/remember-1.9/work/remember-1.9/remember-experimental.el:
  !! File error (("Cannot open load file" "planner"))
Done
While compiling toplevel forms in file /var/tmp/portage/remember-1.9/work/remember-1.9/remember-planner.el:
  !! File error (("Cannot open load file" "planner"))
Done
makeinfo remember-el.texi
>>> Source compiled.

This does not stop remember from being emerged, and it seems to work. Are these optional dependencies or something?
Comment 7 Josh Glover (RETIRED) gentoo-dev 2006-08-11 23:36:32 UTC
GNU Emacs Herd: I'll be happy to take this bug, as I am helping Evan with his first ebuild submission. Let me know if you guys are cool with this.
Comment 8 Evan Monroig 2006-08-14 02:04:58 UTC
(In reply to comment #6)
> No need to set S manually to the default. :)

Thanks for the correction
 
> Now, here is what I get when I emerge it:
> 
[snip]
>
> This does not stop remember from being emerged, and it seems to work. Are these
> optional dependencies or something?

Yes, I had some of those also, although less than you because I have more app-emacs/... packages installed.

remember-el works with several backends and can make back-links to several kind of buffers, so I think that these are optional dependencies, and we don't need to satisfy them by default.

But I wonder what happens if we install one of those dependencies later and expect it to work together with remember-el.

I just sent an email to check with the upstream mailing list.
Comment 9 Evan Monroig 2006-08-15 16:56:27 UTC
I posted to the remember mailing list, and the current maintainer Michael Olson replied that they are all optional dependences [1].

Would the best approach be to generate separate packages?

[1] https://mail.gna.org/public/remember-el-discuss/2006-08/msg00008.html
Comment 10 Josh Glover (RETIRED) gentoo-dev 2006-08-19 05:14:32 UTC
Created attachment 94605 [details]
New version of remember-1.9.ebuild; added local USE flags for optional dependencies

No, there is no need to create separate packages. This is a perfect application for local USE variables. See:

http://www.gentoo.org/proj/en/devrel/handbook/handbook.xml?part=2&chap=1#doc_chap2_sect5

However, there are not ebuilds for emacs-wiki-blosxom, emacs-wiki-journal, or app-emacs/url in Portage yet. Evan, your mission, should you choose to accept it, is to create ebuilds for each of them, so we can add three more local USE flags and thus provide support for all of remember's optional dependencies.
Comment 11 Josh Glover (RETIRED) gentoo-dev 2006-08-19 05:15:57 UTC
Created attachment 94606 [details, diff]
Patch to use.local.desc containing new local USE flags
Comment 12 Josh Glover (RETIRED) gentoo-dev 2006-08-19 05:17:04 UTC
I have not heard any complaints from the GNU Emacs herd, so I am taking this bug.
Comment 13 Josh Glover (RETIRED) gentoo-dev 2006-08-19 05:17:25 UTC
Accepting bug.
Comment 14 Josh Glover (RETIRED) gentoo-dev 2006-08-31 17:11:42 UTC
Ebuild commited to Portage CVS.