Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 271270 - dev-util/android-sdk bundles android.el but no emacs USE flag
Summary: dev-util/android-sdk bundles android.el but no emacs USE flag
Status: RESOLVED UPSTREAM
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: High enhancement
Assignee: Krzysztof Pawlik (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 184166
  Show dependency tree
 
Reported: 2009-05-26 06:56 UTC by Gabor Garami
Modified: 2009-07-09 16:45 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Gabor Garami 2009-05-26 06:56:29 UTC
Android SDK has an emacs support stuff, but no emacs USE flag here. I think it can be useful to emacs users
Comment 1 Christian Faulhammer (RETIRED) gentoo-dev 2009-06-03 09:46:30 UTC
Gabor, have you used the Emacs support?

It seems rather basic and has no autoload magic, so a hard-coded load is a no-go for us (Emacs team).  It seems to work, but more than byte-compiling and installing in /usr/share/emacs/site-lisp is not in our scope.
Comment 2 Gabor Garami 2009-06-09 12:51:34 UTC
And no way to do wrapper for it to fit in gentoo-style? 
I currently not using this, because my phone isn't arrived yet, but I thinking on using, because NetBeans is too bloated for a small, fast apps; and i need a hack to make android support correct. This was a basic idea, why I asked this. 
I can include this .el as well as described way on project's homepage, but an official support is always better than own hacks :-).

(In reply to comment #1)
> Gabor, have you used the Emacs support?
> 
> It seems rather basic and has no autoload magic, so a hard-coded load is a
> no-go for us (Emacs team).  It seems to work, but more than byte-compiling and
> installing in /usr/share/emacs/site-lisp is not in our scope.
> 

Comment 3 Christian Faulhammer (RETIRED) gentoo-dev 2009-06-09 12:54:42 UTC
I talked to ulm (the rest of the GNU Emacs team), and we can provide a custom-made autoload for android.  So a USE=emacs is in reach (not today, though). :)
Comment 4 Ulrich Müller gentoo-dev 2009-06-09 12:57:57 UTC
(In reply to comment #2)
> And no way to do wrapper for it to fit in gentoo-style? 

Well, the .el contains only a handful of interactive functions, so there's always the possibility to autoload some (or all) of them.

nelchael, rich0: O.K. with you if the Emacs team is taking care of this issue?
Comment 5 Richard Freeman gentoo-dev 2009-06-09 13:47:41 UTC
No objections here - they have a bit more experience with emacs than I do.  If any changes are needed on the android side let us know so that we can make sure not to disrupt them in the future.
Comment 6 Ulrich Müller gentoo-dev 2009-07-09 16:45:12 UTC
Byte-compilation of android.el fails:

In android-read-project-root:
android.el:41:8:Warning: `(read-directory nil (read-file-name "Android project
    root: " android-project-root nil t nil))' is a malformed function
android.el:41:8:Warning: `(entered-root (read-directory) (read-directory))' is
    a malformed function
android.el:41:8:Warning: `(and entered-root (file-directory-p entered-root))'
    is a malformed function
android.el:57:16:Warning: reference to free variable `entered-root'

In android-uninstall-app:
android.el:104:40:Warning: reference to free variable `package'

In end of data:
android.el:132:1:Warning: the following functions are not known to be defined:
    labels, do

Most of these warnings are probably fixable by adding "(require 'cl)" or "(eval-when-compile (require 'cl))" at the top of android.el. However, since we are newly adding Emacs support for the Gentoo package, I am of the opinion that upstream should fix their code before we go ahead.