-
Notifications
You must be signed in to change notification settings - Fork 151
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Byte compilation issue with straight.el at commit b3760f5 #1136
Labels
Comments
I'm unable to reproduce with the following test case: Test Case(straight-bug-report
:post-bootstrap
(straight-use-package '(one :host github :repo "tonyaldon/one.el"))
(one-default-new-project)
(one-build))
OutputBootstrapping straight.el...
Bootstrapping straight.el...done
Looking for gnu-elpa-mirror recipe → Cloning melpa...
Looking for gnu-elpa-mirror recipe → Cloning melpa...done
Looking for nongnu-elpa recipe → Cloning gnu-elpa-mirror...
Looking for nongnu-elpa recipe → Cloning gnu-elpa-mirror...done
Looking for emacsmirror-mirror recipe → Cloning nongnu-elpa...
Looking for emacsmirror-mirror recipe → Cloning nongnu-elpa...done
Looking for emacsmirror-mirror recipe → Cloning el-get...
Looking for emacsmirror-mirror recipe → Cloning el-get...done
Looking for straight recipe → Cloning emacsmirror-mirror...
Looking for straight recipe → Cloning emacsmirror-mirror...done
Building straight...
Building straight...done
Test run with version: prerelease (HEAD -> develop, origin/master, origin/develop, origin/HEAD) b3760f5 2023-10-21
Cloning one.el...
Cloning one.el...done
Building one...
Building one → Cloning jack...
Building one → Cloning jack...done
Building one → Building jack...
Building one → Building jack...done
Building one → Cloning emacs-htmlize...
Building one → Cloning emacs-htmlize...done
Building one → Building htmlize...
Building one → Building htmlize...done
Building one...
Building one...done
Build pages...
Build page ‘/’
Build page ‘/blog/default-home-list-pages/’
Build page ‘/blog/default/’
Setting up indent for shell type bash
Indentation variables are now local.
Indentation setup for shell type bash
Build page ‘/blog/one-default-with-toc/’
Setting up indent for shell type bash
Indentation variables are now local.
Indentation setup for shell type bash
Build page ‘/blog/one-default-with-sidebar/’
Setting up indent for shell type bash
Indentation variables are now local.
Indentation setup for shell type bash
Build page ‘/blog/one-default-doc/’
Setting up indent for shell type bash
Indentation variables are now local.
Indentation setup for shell type bash
Build pages...done
Packages:
"straight" n/a develop b3760f5 2023-10-21
"org-elpa" n/a n/a
"melpa" n/a master 60b3f2b1 2023-12-17
"gnu-elpa-mirror" n/a master ab9bcb0 2023-12-18
"nongnu-elpa" n/a main b299b0d 2023-11-23
"el-get" melpa master a5ba5852 2023-10-20
"emacsmirror-mirror" n/a master 49cd2c4 2023-12-12
"one" melpa master f94f183 2023-12-19
"jack" melpa master 77f51e6 2023-11-12
"htmlize" melpa master dd27bc3 2021-08-25
Does the error occur for you if you evaluate that test case? |
I just evaluated the test case and no error occurred. Test Case(straight-bug-report
:post-bootstrap
(straight-use-package
'(one :host github :repo "tonyaldon/one.el"))
(one-default-new-project)
(one-build))
OutputBootstrapping straight.el...
Bootstrapping straight.el...done
Looking for gnu-elpa-mirror recipe → Cloning melpa...
Looking for gnu-elpa-mirror recipe → Cloning melpa...done
Looking for nongnu-elpa recipe → Cloning gnu-elpa-mirror...
Looking for nongnu-elpa recipe → Cloning gnu-elpa-mirror...done
Looking for emacsmirror-mirror recipe → Cloning nongnu-elpa...
Looking for emacsmirror-mirror recipe → Cloning nongnu-elpa...done
Looking for emacsmirror-mirror recipe → Cloning el-get...
Looking for emacsmirror-mirror recipe → Cloning el-get...done
Looking for straight recipe → Cloning emacsmirror-mirror...
Looking for straight recipe → Cloning emacsmirror-mirror...done
Building straight...
Building straight...done
Test run with version: prerelease (HEAD -> develop, origin/master, origin/develop, origin/HEAD) b3760f5 2023-10-21
Cloning one.el...
Cloning one.el...done
Building one...
Building one → Cloning jack...
Building one → Cloning jack...done
Building one → Building jack...
Building one → Building jack...done
Building one → Cloning emacs-htmlize...
Building one → Cloning emacs-htmlize...done
Building one → Building htmlize...
Building one → Building htmlize...done
Building one...
Building one...done
Build pages...
Build page ‘/’
Build page ‘/blog/default-home-list-pages/’
Build page ‘/blog/default/’
Setting up indent for shell type bash
Indentation variables are now local.
Indentation setup for shell type bash
Build page ‘/blog/one-default-with-toc/’
Setting up indent for shell type bash
Indentation variables are now local.
Indentation setup for shell type bash
Build page ‘/blog/one-default-with-sidebar/’
Setting up indent for shell type bash
Indentation variables are now local.
Indentation setup for shell type bash
Build page ‘/blog/one-default-doc/’
Setting up indent for shell type bash
Indentation variables are now local.
Indentation setup for shell type bash
Build pages...done
Packages:
"straight" n/a develop b3760f5 2023-10-21
"org-elpa" n/a n/a
"melpa" n/a master 60b3f2b1 2023-12-17
"gnu-elpa-mirror" n/a master ab9bcb0 2023-12-18
"nongnu-elpa" n/a main b299b0d 2023-11-23
"el-get" melpa master a5ba5852 2023-10-20
"emacsmirror-mirror" n/a master 49cd2c4 2023-12-12
"one" melpa master f94f183 2023-12-19
"jack" melpa master 77f51e6 2023-11-12
"htmlize" melpa master dd27bc3 2021-08-25
|
See if you can add to that reproduction case in a way that reproduces the error. |
Thanks for looking at it. I'll try to see how can reproduce the error within the test case. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
What's wrong
Byte compilation done by straight differs from the one done by
byte-compile-file
and seems to create a symbol that doesn't exist.Directions to reproduce
When I let
straight
install and build the packageone.el
like thisand called the following commands sequentially
I got the following error:
It is normal because
org-element--property
is not defined byorg-mode
. But what caught my attention is thatone.el
never callsorg-element--property
, it only callsorg-element-property
which is well defined.I found
org-element--property
symbol instraight/build/one/one.elc
file generated bystraight
(if I understand correctly):But if I byte compile
one.el
file withbyte-compile-file
directly, I no longer have that issue and the compiled file looks like this:I recently switched from straight from commit 3eca39d to straight at commit b3760f5. I hadn't that issue with 3eca39d.
So, I disable byte compilation for that package:
What do you think? Am I doing something wrong?
Version information
The text was updated successfully, but these errors were encountered: