-
-
Notifications
You must be signed in to change notification settings - Fork 27
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
QT version 5.5 or 5.15? #331
Comments
Attaching log |
Interesting. Qt 5.15 should be fine. This largely looks like perhaps there is a compiler issue of some kind. It is complaining about what appears to be valid constructs so maybe it's not detecting that it can enable C++14 mode. I'll look at it after the weekend. |
Thanks, I was hoping to get a chance to look at this more too the weekend, but we'll see. |
I have nedit-ng (commit d417b1e) built against Qt 5.15.3 from Homebrew on an M1 MacBook Pro. One thing I did discover is that having Qt6 installed as well can prevent it building properly against Qt5, so I recommend uninstalling Qt6 and trying again. |
I want to say I set the env variables to use QT5 , with just QT6 I didn't even get that far |
Do you still have Qt6 installed though? I think I had to uninstall it for the build to work, even when pointing to Qt5. Worth trying if you can (I might have had to reinstall Qt5 afterwards as well to ensure soft-links were pointing to the right places, but there might be simpler ways to do that if you know Homebrew better than I do). |
Ill def try it soon |
Had a bit of time , removed qt6 and it seems I get some different errors now: In file included from /opt/homebrew/include/boost/type_traits/is_function.hpp:19: |
What version of Bison are you using? The one provided by Apple no longer works with recent nedit-ng versions, so you may need to install a newer one with Homebrew. IIRC that was causing problems in the Interpreter build since that's what was using the new stuff, and that's where your errors seem to be coming from. Have you read the latest parts of the Wiki page about building on MacOS? I added the Makefile there which is how I do my builds, and I recommend trying that. The only lines you should need to adjust would be the settings for HOMEBREW and PREFIX. |
I didn't see the mac makefile, but I did just try it and didn't have any luck. I do have the homebrew bison: bison --version Copyright (C) 2021 Free Software Foundation, Inc. which bison Makefile was added to nedit-ng/Makefile still similar issues when trying to use just the Makefile In file included from /opt/homebrew/include/boost/type_traits/is_function.hpp:19: |
I actually got further, i had to nuke my build directory, replace the makefile with the one suggested, and re-follow the instructions: [100%] Linking CXX executable ../nedit-import Interesting that libX11 is still needed for this port? |
Which version of nedit-ng are you building? It doesn't look like it has been needed since February, so try building the tip of the master branch:
|
Hmm, I just did git clone from the master branch code I believe |
Hmm, I just tried a build from scratch and now I'm also getting that build failure:
It looks like I disabled building the diff --git a/CMakeLists.txt b/CMakeLists.txt
index b6b8846d..1bcd8de6 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -76,6 +76,6 @@ add_subdirectory(Interpreter)
if(UNIX)
find_package(X11)
if(${X11_FOUND})
- add_subdirectory(import)
+# add_subdirectory(import)
endif()
endif() If you need to import settings from older configuration files you might have to do that part on a Linux system that has nedit-import build working. If not this should be sufficient to get a working editor. |
Thanks for looking into this guy's. I've been stuck in an airport the past 24 hours but once I finally get home we'll get you all sorted out! I think the X11 stuff is an easy fix 😁 |
Awesome this worked now! I got it to build between the Makefile and removal of the libX11 part. Oddly enough I had libX11 from homebrew but it didn't seem like it liked it, for whatever reason. |
Random but i recall the nedit of days old has a motif like scheme / color to it, anyone recall the background/foreground hex codes? |
@profallgood It's probably more appropriate to ask that in the Discussions area. The subject line here doesn't match the final problem, but @eteran may use this issue as a reminder to fix the macOS build problem. |
@profallgood I can certainly look them up when I'm back at my comp. It's in the git history for sure since I didn't change the color scheme right away. |
Fair enough, was just a random thought, I can dig around for it later, but yea hopefully this can stay open as a reminder and then the build problem gets fixed, assuming that's the case. |
@profallgood OK, so to get a more "classic" color scheme, you can use the following
I can say that this will basically only apply to the text area. The Qt widgets and window treatment has its own theming that is highly editable via a |
Trying to build this on my arm mac and having some difficulty. It seems you mention 5.5 as the min QT version but in homebrew it goes up to 5.15. I also check qt's main website and they don't have a version 5.5.. just 5.15, then on to 6.3.0..
Here's just the beginning snippet:
In file included from /Users/nick/nedit-ng/Regex/Regex.cpp:80:
In file included from /Users/nick/nedit-ng/Regex/Regex.h:6:
In file included from /Users/nick/nedit-ng/Regex/RegexError.h:5:
In file included from /Users/nick/nedit-ng/Util/include/Util/Compiler.h:5:
In file included from /opt/homebrew/Cellar/qt@5/5.15.3/lib/QtCore.framework/Headers/QtGlobal:1:
/opt/homebrew/Cellar/qt@5/5.15.3/lib/QtCore.framework/Headers/qglobal.h:399:11: error: 'maybe_unused' attribute cannot be applied to types
constexpr Q_DECL_UNUSED Deprecated_t Deprecated = {};
^
In file included from /Users/nick/nedit-ng/Regex/Regex.cpp:80:
In file included from /Users/nick/nedit-ng/Regex/Regex.h:6:
In file included from /Users/nick/nedit-ng/Regex/RegexError.h:5:
In file included from /Users/nick/nedit-ng/Util/include/Util/Compiler.h:5:
In file included from /opt/homebrew/Cellar/qt@5/5.15.3/lib/QtCore.framework/Headers/QtGlobal:1:
/opt/homebrew/Cellar/qt@5/5.15.3/lib/QtCore.framework/Headers/qglobal.h:399:11: error: 'maybe_unused' attribute cannot be applied to types
constexpr Q_DECL_UNUSED Deprecated_t Deprecated = {};
^
/opt/homebrew/include/QtCore/qcompilerdetection.h:1100:27: note: expanded from macro 'Q_DECL_UNUSED'
define Q_DECL_UNUSED [[maybe_unused]]
The text was updated successfully, but these errors were encountered: