Re: [NEW] btop++

2024-04-16 Thread Stuart Henderson
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++

2024-04-15 Thread Robert Palm



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++

2024-04-08 Thread 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++

2024-04-08 Thread Laurent Cheylus
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++

2024-04-08 Thread Stuart Henderson
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++

2024-04-06 Thread Jos Dehaes
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++

2023-09-06 Thread Stuart Henderson
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++

2023-09-05 Thread Stuart Henderson
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++

2023-09-05 Thread Stuart Henderson
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++

2023-09-05 Thread Jos Dehaes
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