-
Notifications
You must be signed in to change notification settings - Fork 270
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
2.0.0: build fails end compile time warnings #1303
Comments
Looks like number of compile time warnings is quite high $ rpmbuild -ba geany-plugins.spec --quiet 2>&1 | grep -- \\[-W | sed 's/.*\[//; s/\]//' | sort | uniq -c | sort -nr
613 -Wdeprecated-declarations
11 -Wunused-variable
11 -Wimplicit-function-declaration
8 -Wunused-result
7 -Wdiscarded-qualifiers
6 -Wunused-function
5 -Wformat=
1 -Wunused-const-variable=
1 -Wunused-but-set-variable
1 -Wincompatible-pointer-types |
I'm using gcc 14.x so it shows a bit more like Extracted stderr with all warnings and errors geany-plugins.stderr.txt |
Build may fail in lua beause I'm using patch which enabled build with lua >= 5.2 --- a/build/geanylua.m4
+++ b/build/geanylua.m4
@@ -14,10 +14,8 @@
done])
LUA_VERSION=5.1
- LUA_VERSION_BOUNDARY=5.2
GP_CHECK_PLUGIN_DEPS([GeanyLua], [LUA],
- [${LUA_PKG_NAME} >= ${LUA_VERSION}
- ${LUA_PKG_NAME} < ${LUA_VERSION_BOUNDARY}])
+ [${LUA_PKG_NAME} >= ${LUA_VERSION}])
GP_CHECK_PLUGIN_DEPS([GeanyLua], [GMODULE], [gmodule-2.0])
GP_COMMIT_PLUGIN_STATUS([GeanyLua]) It would be good to update for latest lua because older versions are buggy and longer maintained (there are few CVEs in lua older than 5.4). |
GTK deprecations should continue to work on GTK3 which is all Geany and plugins support. So these warnings will likely not go away until a GTK4 port happens, so they should not be allowed fail a build. For the Geanylua plugin, are you sure your build is using a version of the lua headers/library that is supported by the plugin? Note that geanylua is unmaintained and will not have been updated if type and function names in Lua headers have changed. Unless "somebody" with the time and ability takes the plugin under their wing it won't be updated to newer versions. The project organiser incompatible pointer just needs an explicit cast @techee (As an aside, I thought incompatible pointers was in GCC 13, not 14) |
After disable build projectorganizer and geanylua I was able to build geany-plugins.
As I wrote I've been trying to build against latest lua 5.4 using
gcc 14.x is now even more sensitive on pointer typing. |
Yeah, I didn't see your post until I had hit comment
As a C++ programmer I totally approve (he says writing |
This happened in #1235. |
Well, there's also the competing #1238 that tries to support about everything. I haven't checked if it works, I personally would prefer supporting only a single version since nobody will be able to test all the versions (especially when the plugin is unmaintained). |
Looks like latest version build fails
The text was updated successfully, but these errors were encountered: