Comments (9)
Dave, any chance you beat me to fix this? I'll have a look in the evening...
from el-get.
BTW, my el-get config is here: https://github.com/purcell/emacs.d/blob/master/init-el-get.el
from el-get.
Ok my first thinking reading your el-get-sources was to get them all in:
(let ((n (remove nil
(mapcar (lambda (s)
(unless
(file-exists-p (format "~/dev/emacs/el-get/recipes/%s.el" (el-get-source-name s))) s))
(loop for r in purcell-el-get-sources when (not (symbolp r)) collect r)))))
(loop for s in n
do (with-temp-file (format "~/temp/purcell-el-get-recipes/%s.el" (el-get-source-name s))
(insert (prin1-to-string s)))))
Now, Dave, are you reading and having some time to look at the autoload issue? :)
from el-get.
- I'm reading
- Don't have much time but since there's claimed to be a bug in my code I will try
- I don't understand the nature of the problem. If there's an "offending autoload" then isn't this a problem in the
diff-git
package? - If someone could explain what they understand so far about the problem it might help
from el-get.
I think it's explained quite well above -- note that the problem is not specific to the diff-git recipe.
Essentially, with this set of el-get recipes, start-up is fine unless the autoloads have already been generated, at which point errors like the above happen.
Now, I've (since) hacked el-get locally to disable the autoload functionality completely, but I'm happy to help track down what's wrong; it'd just be good to get some pointers.
from el-get.
Could we get a defvar el-get-manage-autoload (or a better name) in the main el-get, so that users can disable the feature? I'm not sure if we want to add a property to be able to do that per package like with :compile and el-get-byte-compile, though.
from el-get.
@purcell: Can you produce a minimal (preferably runnable via emacs -Q) example that demonstrates this problem? If so, I can probably fix it quickly.
@dimitri: I'd rather not address a bug by giving users the option to turn off the feature.
from el-get.
Okay, so with the latest el-get I no longer get the issue when I turn off my disable-autoloads hack. I do, however, get a problem with session.el (http://www.emacswiki.org/emacs/session.el) -- it results in a loaddefs file containing (defunx session-initialize ...)
, where defunx is a non-autoloaded macro defined in session.el
... yuck! But that's not an el-get issue, since el-get's just calling the usual update-directory-autoloads
function.
So I'll go ahead and close this issue.
Regarding the option to disable autoloads, I agree this would have been the wrong way to tackle this issue, but I'd like such an option anyway. At the very least, being able to define :autoload nil
or similar in a recipe would allow the handling of libraries like session.el
which work perfectly well when require
d, but fail when autoloaded.
from el-get.
I'd like to be able to disable autoloads too, for giving the choice to users. That's independant of whether there's bugs in the autoloads code or not.
from el-get.
Related Issues (20)
- package-archive-priorities ignored when specifying :repo with type: elpa
- Compilation fix for Emacs 28 native-comp branch breaks build for all the other Emacs versions HOT 1
- Byte compilation should be separately done for each file
- Usage of 'go get' for recipes of type "go" is broken with golang 1.16+
- company-mode.rcp out of date HOT 1
- (el-get-bundle elpa:magit) fails to resolve correct dependencies
- Using ~/.config/emacs instead of ~/.emacs.d HOT 3
- el-get from melpa does not include info documentation for el-get. WTF?
- Continuous integration for el-get packages HOT 6
- adoc-mode has moved/been adopted
- Unable to activate lsp-docker
- better-defaults repo url is invalid
- Incorrect recipe for git-timemachine HOT 1
- Recipe for scratch package is out of date
- Depends deprecated `autoload` package HOT 1
- missing :files directive in systemd-mode recipe
- mastodon recipe points to incorrect repository
- el-get-self-update fails to update (melpa directory)
- Problem when installing el-get on emacs 29.1 HOT 6
- Sync raises error in Emacs 29.1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from el-get.