-
Notifications
You must be signed in to change notification settings - Fork 12
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
Server Won't Start #3
Comments
Hm, we should be (gracefully) handling that Thanks for the report! It's on the docket. |
Might be worth trying with the same version as it's built with? https://github.com/softwareCobbler/luceedebug/blob/master/luceedebug/extern/5.3.9.158-SNAPSHOT.lco |
It's not "fixed" but pushed a change so that it doesn't crash in this location on I'd be interested to know which files are missing line info; the new trace output might help shed light on that. Assuming they are Lucee generated class files, I thought source line info was always emitted into the generated class files (and rely on that to be the case to instrument stepping). @zspitzer I won't rule out a version conflict; but, the error seems correlated with asking for line information against a |
@softwareCobbler , thanks for the semi-fix. Here's where I am now:
|
Ah, that's rough. After instrumentation, the method is too large (per classfile docs it's a u4 (4 bytes <==> 32 bit)). A little googling seems to indicate that instrumentation is a common cause of this error. Pushed a workaround to disable instrumentation for the affected file, which unfortunately means breakpoints and frame info simply won't work for methods in that file. It will print a message indicating the name of the affected classfile when this happens. We could probably do better and either:
|
Thanks, I'll give it a shot later. Also, could you elaborate on the u32 comment? |
well, they say u4, which they're using to mean 4 bytes; I was using u32 meaning 32 bits ... same same. But now I've gone and confused myself, since a 32bit number is what, 4GB? There's no way a method is getting to that size. Oh ... it's a 32 bit number, but then they go and say
So in effect a 16 bit number. |
Server starts!
Ready to close, unless you've got any questions for me. |
@softwareCobbler Lucee has support for splitting large class files when they are too large |
I'm using the lucee docker image (
lucee/lucee:5.3.7.47
). I'm new to luceedebug so I have no idea what I'm doing wrong, but the server won't start.You can see the
agentlib
andjavaagent
configs I'm trying to use in the logs, below.Any ideas?
The text was updated successfully, but these errors were encountered: