Re: [NEW] btop++
On 2024/04/16 08:52, Robert Palm wrote: > > Morning ports > > did this made it into the source tree, already ? No (as you can see for yourself). It needs an ok from a developer to import.
Re: [NEW] btop++
Morning ports did this made it into the source tree, already ? Thanks. Quoting Jos Dehaes : Thanks Stuart! This works. Can it be included in the ports tree now? Cheers, Jos On Mon, 8 Apr 2024 at 17:07, Stuart Henderson wrote: Attached. On 2024/04/06 21:57, Jos Dehaes wrote: > Hi, > > With the 7.5 release, I tried again, and finally btop now builds with attached port. However, > I'm struggling with the PLIST file, it seems the fake install is messing up the prefix > somehow. > > Can someone help? > > Cheers, > Jos > > On Tue, 5 Sept 2023 at 19:59, Stuart Henderson wrote: > > On 2023/09/05 15:58, Stuart Henderson wrote: > > On 2023/09/05 16:34, Jos Dehaes wrote: > > > Hi, > > > > > > I want to add the btop port. I used a package from the tech mailing list by > > > Stuart Henderson as a base. The problem is that btop needs to be compiled > > > with C++20 compiler. He suggested using clang 16, but I could not get that > > > to work. > > > > > > I just used the native g++ detection in the btop Makefile, and this works, > > > provided that g++ version 11 is installed. I could not find a way to encode > > > that in the port. > > > > > > Kindly provide guidance on how to get this included. > > > > > > Cheers, > > > Jos > > > > As I said before, GCC 11 cannot be used as a build dependency in ports. > > > > The clang 16 bits in the tree are very new, wait for it to settle a bit. > > > > the attached tar does build with clang 16 but it still fails: > > c++ -std=c++20 -pthread -D_FORTIFY_SOURCE=2 -D_GLIBCXX_ASSERTIONS -D_FILE_OFFSET_BITS=64 > -fexceptions -fstack-clash-protection -fcf-protection -fstack-protector -lkvm -O2 > -ftree-vectorize -flto=thin -Wall -Wextra -pedantic -isystem include -Isrc -MMD -c -o obj/ > btop.o src/btop.cpp > clang-16: warning: -lkvm: 'linker' input unused [-Wunused-command-line-argument] > clang-16: warning: argument unused during compilation: '-fstack-clash-protection' > [-Wunused-command-line-argument] > In file included from src/btop.cpp:47: > src/btop_tools.hpp:159:48: error: no member named 'count_if' in namespace 'std::ranges' > return (wide ? wide_ulen(str) : std::ranges::count_if(str, [](char c) { > return (static_cast(c) & 0xC0) != 0x80; })); > ~^ > src/btop_tools.hpp:179:16: error: no member named 'for_each' in namespace 'std::ranges' > std::ranges::for_each(str, [](auto& c) { c = ::toupper(c); } ); > ~^ > src/btop_tools.hpp:185:16: error: no member named 'for_each' in namespace 'std::ranges' > std::ranges::for_each(str, [](char& c) { c = ::tolower(c); } ); > ~^ > src/btop_tools.hpp:214:10: error: no member named 'distance' in namespace 'std::ranges'; > did you mean 'std::distance'? > return std::ranges::distance(vec.begin(), std::ranges::find(vec, > find_val)); >^ >std::distance > /usr/include/c++/v1/__iterator/distance.h:47:1: note: 'std::distance' declared here > distance(_InputIter __first, _InputIter __last) > ^ > 4 errors generated. > >
Re: [NEW] btop++
Thanks Stuart! This works. Can it be included in the ports tree now? Cheers, Jos On Mon, 8 Apr 2024 at 17:07, Stuart Henderson wrote: > Attached. > > On 2024/04/06 21:57, Jos Dehaes wrote: > > Hi, > > > > With the 7.5 release, I tried again, and finally btop now builds with > attached port. However, > > I'm struggling with the PLIST file, it seems the fake install is messing > up the prefix > > somehow. > > > > Can someone help? > > > > Cheers, > > Jos > > > > On Tue, 5 Sept 2023 at 19:59, Stuart Henderson > wrote: > > > > On 2023/09/05 15:58, Stuart Henderson wrote: > > > On 2023/09/05 16:34, Jos Dehaes wrote: > > > > Hi, > > > > > > > > I want to add the btop port. I used a package from the tech > mailing list by > > > > Stuart Henderson as a base. The problem is that btop needs to be > compiled > > > > with C++20 compiler. He suggested using clang 16, but I could > not get that > > > > to work. > > > > > > > > I just used the native g++ detection in the btop Makefile, and > this works, > > > > provided that g++ version 11 is installed. I could not find a > way to encode > > > > that in the port. > > > > > > > > Kindly provide guidance on how to get this included. > > > > > > > > Cheers, > > > > Jos > > > > > > As I said before, GCC 11 cannot be used as a build dependency in > ports. > > > > > > The clang 16 bits in the tree are very new, wait for it to settle > a bit. > > > > > > > the attached tar does build with clang 16 but it still fails: > > > > c++ -std=c++20 -pthread -D_FORTIFY_SOURCE=2 -D_GLIBCXX_ASSERTIONS > -D_FILE_OFFSET_BITS=64 > > -fexceptions -fstack-clash-protection -fcf-protection > -fstack-protector -lkvm -O2 > > -ftree-vectorize -flto=thin -Wall -Wextra -pedantic -isystem include > -Isrc -MMD -c -o obj/ > > btop.o src/btop.cpp > > clang-16: warning: -lkvm: 'linker' input unused > [-Wunused-command-line-argument] > > clang-16: warning: argument unused during compilation: > '-fstack-clash-protection' > > [-Wunused-command-line-argument] > > In file included from src/btop.cpp:47: > > src/btop_tools.hpp:159:48: error: no member named 'count_if' in > namespace 'std::ranges' > > return (wide ? wide_ulen(str) : > std::ranges::count_if(str, [](char c) { > > return (static_cast(c) & 0xC0) != 0x80; })); > > ~^ > > src/btop_tools.hpp:179:16: error: no member named 'for_each' in > namespace 'std::ranges' > > std::ranges::for_each(str, [](auto& c) { c = > ::toupper(c); } ); > > ~^ > > src/btop_tools.hpp:185:16: error: no member named 'for_each' in > namespace 'std::ranges' > > std::ranges::for_each(str, [](char& c) { c = > ::tolower(c); } ); > > ~^ > > src/btop_tools.hpp:214:10: error: no member named 'distance' in > namespace 'std::ranges'; > > did you mean 'std::distance'? > > return std::ranges::distance(vec.begin(), > std::ranges::find(vec, > > find_val)); > >^ > >std::distance > > /usr/include/c++/v1/__iterator/distance.h:47:1: note: > 'std::distance' declared here > > distance(_InputIter __first, _InputIter __last) > > ^ > > 4 errors generated. > > > > > > >
Re: [NEW] btop++
Hi Jos, On Sat, 6 Apr 2024 21:57:30 +0200, Jos Dehaes wrote: > With the 7.5 release, I tried again, and finally btop now builds with > attached port. However, I'm struggling with the PLIST file, it seems the > fake install is messing up the prefix somehow. I have already submitted a sysutils/btop port for version 1.3.0 => see this thread with some comments from sthen@ https://marc.info/?l=openbsd- ports&m=170635392519643&w=2 I need to update my port for version 1.3.2 and finalize some PRs upstream to modify Makefile according my patch (no verbose, don't use all CPU by default during compilation...). Laurent
Re: [NEW] btop++
Attached. On 2024/04/06 21:57, Jos Dehaes wrote: > Hi, > > With the 7.5 release, I tried again, and finally btop now builds with > attached port. However, > I'm struggling with the PLIST file, it seems the fake install is messing up > the prefix > somehow. > > Can someone help? > > Cheers, > Jos > > On Tue, 5 Sept 2023 at 19:59, Stuart Henderson wrote: > > On 2023/09/05 15:58, Stuart Henderson wrote: > > On 2023/09/05 16:34, Jos Dehaes wrote: > > > Hi, > > > > > > I want to add the btop port. I used a package from the tech mailing > list by > > > Stuart Henderson as a base. The problem is that btop needs to be > compiled > > > with C++20 compiler. He suggested using clang 16, but I could not get > that > > > to work. > > > > > > I just used the native g++ detection in the btop Makefile, and this > works, > > > provided that g++ version 11 is installed. I could not find a way to > encode > > > that in the port. > > > > > > Kindly provide guidance on how to get this included. > > > > > > Cheers, > > > Jos > > > > As I said before, GCC 11 cannot be used as a build dependency in ports. > > > > The clang 16 bits in the tree are very new, wait for it to settle a bit. > > > > the attached tar does build with clang 16 but it still fails: > > c++ -std=c++20 -pthread -D_FORTIFY_SOURCE=2 -D_GLIBCXX_ASSERTIONS > -D_FILE_OFFSET_BITS=64 > -fexceptions -fstack-clash-protection -fcf-protection -fstack-protector > -lkvm -O2 > -ftree-vectorize -flto=thin -Wall -Wextra -pedantic -isystem include > -Isrc -MMD -c -o obj/ > btop.o src/btop.cpp > clang-16: warning: -lkvm: 'linker' input unused > [-Wunused-command-line-argument] > clang-16: warning: argument unused during compilation: > '-fstack-clash-protection' > [-Wunused-command-line-argument] > In file included from src/btop.cpp:47: > src/btop_tools.hpp:159:48: error: no member named 'count_if' in namespace > 'std::ranges' > return (wide ? wide_ulen(str) : > std::ranges::count_if(str, [](char c) { > return (static_cast(c) & 0xC0) != 0x80; })); > ~^ > src/btop_tools.hpp:179:16: error: no member named 'for_each' in namespace > 'std::ranges' > std::ranges::for_each(str, [](auto& c) { c = > ::toupper(c); } ); > ~^ > src/btop_tools.hpp:185:16: error: no member named 'for_each' in namespace > 'std::ranges' > std::ranges::for_each(str, [](char& c) { c = > ::tolower(c); } ); > ~^ > src/btop_tools.hpp:214:10: error: no member named 'distance' in namespace > 'std::ranges'; > did you mean 'std::distance'? > return std::ranges::distance(vec.begin(), > std::ranges::find(vec, > find_val)); > ^ > std::distance > /usr/include/c++/v1/__iterator/distance.h:47:1: note: 'std::distance' > declared here > distance(_InputIter __first, _InputIter __last) > ^ > 4 errors generated. > > btop.tgz Description: application/tar-gz
Re: [NEW] btop++
Hi, With the 7.5 release, I tried again, and finally btop now builds with attached port. However, I'm struggling with the PLIST file, it seems the fake install is messing up the prefix somehow. Can someone help? Cheers, Jos On Tue, 5 Sept 2023 at 19:59, Stuart Henderson wrote: > On 2023/09/05 15:58, Stuart Henderson wrote: > > On 2023/09/05 16:34, Jos Dehaes wrote: > > > Hi, > > > > > > I want to add the btop port. I used a package from the tech mailing > list by > > > Stuart Henderson as a base. The problem is that btop needs to be > compiled > > > with C++20 compiler. He suggested using clang 16, but I could not get > that > > > to work. > > > > > > I just used the native g++ detection in the btop Makefile, and this > works, > > > provided that g++ version 11 is installed. I could not find a way to > encode > > > that in the port. > > > > > > Kindly provide guidance on how to get this included. > > > > > > Cheers, > > > Jos > > > > As I said before, GCC 11 cannot be used as a build dependency in ports. > > > > The clang 16 bits in the tree are very new, wait for it to settle a bit. > > > > the attached tar does build with clang 16 but it still fails: > > c++ -std=c++20 -pthread -D_FORTIFY_SOURCE=2 -D_GLIBCXX_ASSERTIONS > -D_FILE_OFFSET_BITS=64 -fexceptions -fstack-clash-protection > -fcf-protection -fstack-protector -lkvm -O2 -ftree-vectorize -flto=thin > -Wall -Wextra -pedantic -isystem include -Isrc -MMD -c -o obj/btop.o > src/btop.cpp > clang-16: warning: -lkvm: 'linker' input unused > [-Wunused-command-line-argument] > clang-16: warning: argument unused during compilation: > '-fstack-clash-protection' [-Wunused-command-line-argument] > In file included from src/btop.cpp:47: > src/btop_tools.hpp:159:48: error: no member named 'count_if' in namespace > 'std::ranges' > return (wide ? wide_ulen(str) : std::ranges::count_if(str, > [](char c) { return (static_cast(c) & 0xC0) != 0x80; })); > ~^ > src/btop_tools.hpp:179:16: error: no member named 'for_each' in namespace > 'std::ranges' > std::ranges::for_each(str, [](auto& c) { c = ::toupper(c); > } ); > ~^ > src/btop_tools.hpp:185:16: error: no member named 'for_each' in namespace > 'std::ranges' > std::ranges::for_each(str, [](char& c) { c = ::tolower(c); > } ); > ~^ > src/btop_tools.hpp:214:10: error: no member named 'distance' in namespace > 'std::ranges'; did you mean 'std::distance'? > return std::ranges::distance(vec.begin(), > std::ranges::find(vec, find_val)); >^ >std::distance > /usr/include/c++/v1/__iterator/distance.h:47:1: note: 'std::distance' > declared here > distance(_InputIter __first, _InputIter __last) > ^ > 4 errors generated. > > btop.tgz Description: application/gzip
Re: [NEW] btop++
cc'ing back to ports - On 2023/09/06 13:57, Jos Dehaes wrote: > It's due to the include files. There is no std::ranges::for_each (and the > other ones). > > Also, I notice that clang/c++ is still version 13.0.0 after installing LLVM > 16. the latest version of the port I sent uses clang/llvm 16 on a -current ports tree. you can see that in the output I pasted previously. [...] clang-16: warning: -lkvm: 'linker' input unused [-Wunused-command-line-argument] clang-16: warning: argument unused during compilation: '-fstack-clash-protection' [-Wunused-command-line-argument] In file included from src/btop.cpp:47: src/btop_tools.hpp:159:35: error: use of undeclared identifier 'ranges'; did you mean 'std::ranges'? [...] > Apparently the headers come from comp73 and clang comes from base73 package. > > I did sysupgrade but still the same problem. Until compiler and libc++ are > upgraded, it won't > work with clang. so if it needs new libc++ we just have to park this until libc++ can be updated (this certainly won't be before 7.4 release).
Re: [NEW] btop++
On 2023/09/05 15:58, Stuart Henderson wrote: > On 2023/09/05 16:34, Jos Dehaes wrote: > > Hi, > > > > I want to add the btop port. I used a package from the tech mailing list by > > Stuart Henderson as a base. The problem is that btop needs to be compiled > > with C++20 compiler. He suggested using clang 16, but I could not get that > > to work. > > > > I just used the native g++ detection in the btop Makefile, and this works, > > provided that g++ version 11 is installed. I could not find a way to encode > > that in the port. > > > > Kindly provide guidance on how to get this included. > > > > Cheers, > > Jos > > As I said before, GCC 11 cannot be used as a build dependency in ports. > > The clang 16 bits in the tree are very new, wait for it to settle a bit. > the attached tar does build with clang 16 but it still fails: c++ -std=c++20 -pthread -D_FORTIFY_SOURCE=2 -D_GLIBCXX_ASSERTIONS -D_FILE_OFFSET_BITS=64 -fexceptions -fstack-clash-protection -fcf-protection -fstack-protector -lkvm -O2 -ftree-vectorize -flto=thin -Wall -Wextra -pedantic -isystem include -Isrc -MMD -c -o obj/btop.o src/btop.cpp clang-16: warning: -lkvm: 'linker' input unused [-Wunused-command-line-argument] clang-16: warning: argument unused during compilation: '-fstack-clash-protection' [-Wunused-command-line-argument] In file included from src/btop.cpp:47: src/btop_tools.hpp:159:48: error: no member named 'count_if' in namespace 'std::ranges' return (wide ? wide_ulen(str) : std::ranges::count_if(str, [](char c) { return (static_cast(c) & 0xC0) != 0x80; })); ~^ src/btop_tools.hpp:179:16: error: no member named 'for_each' in namespace 'std::ranges' std::ranges::for_each(str, [](auto& c) { c = ::toupper(c); } ); ~^ src/btop_tools.hpp:185:16: error: no member named 'for_each' in namespace 'std::ranges' std::ranges::for_each(str, [](char& c) { c = ::tolower(c); } ); ~^ src/btop_tools.hpp:214:10: error: no member named 'distance' in namespace 'std::ranges'; did you mean 'std::distance'? return std::ranges::distance(vec.begin(), std::ranges::find(vec, find_val)); ^ std::distance /usr/include/c++/v1/__iterator/distance.h:47:1: note: 'std::distance' declared here distance(_InputIter __first, _InputIter __last) ^ 4 errors generated. btop.tgz Description: application/tar-gz
Re: [NEW] btop++
On 2023/09/05 16:34, Jos Dehaes wrote: > Hi, > > I want to add the btop port. I used a package from the tech mailing list by > Stuart Henderson as a base. The problem is that btop needs to be compiled > with C++20 compiler. He suggested using clang 16, but I could not get that > to work. > > I just used the native g++ detection in the btop Makefile, and this works, > provided that g++ version 11 is installed. I could not find a way to encode > that in the port. > > Kindly provide guidance on how to get this included. > > Cheers, > Jos As I said before, GCC 11 cannot be used as a build dependency in ports. The clang 16 bits in the tree are very new, wait for it to settle a bit.
[NEW] btop++
Hi, I want to add the btop port. I used a package from the tech mailing list by Stuart Henderson as a base. The problem is that btop needs to be compiled with C++20 compiler. He suggested using clang 16, but I could not get that to work. I just used the native g++ detection in the btop Makefile, and this works, provided that g++ version 11 is installed. I could not find a way to encode that in the port. Kindly provide guidance on how to get this included. Cheers, Jos btop.tgz Description: application/compressed-tar