[Cin] Two patches hopefully fixing build on NetBSD 10.0/amd64

Tue Dec 17 00:09:49 CET 2024

I moved the 3 lines in the patch below line: CFG_CFLAGS+=" -pthread -Wall".
But I am getting a syntax error as follows:

./configure: line 17476: syntax error near unexpected token
`-Wno-overloaded-virtual,CFG_CFLAGS'
./configure: line 17476:
`AX_APPEND_COMPILE_FLAGS(-Wno-overloaded-virtual,CFG_CFLAGS)'
make: *** No targets specified and no makefile found.  Stop.

3 lines moved were:
AC_LANG_PUSH([C++])
AX_APPEND_COMPILE_FLAGS([-Wno-overloaded-virtual],[CFG_CFLAGS])
AC_LANG_POP([C++])

On Mon, Dec 16, 2024 at 2:40 AM Andrew Randrianasulu <
randrianasulu at gmail.com> wrote:

>
>
> чт, 28 нояб. 2024 г., 01:54 Andrew Randrianasulu <randrianasulu at gmail.com
> >:
>
>>
>>
>> чт, 28 нояб. 2024 г., 01:47 Phyllis Smith <phylsmith2017 at gmail.com>:
>>
>>> autoconf-archive was already installed;
>>> Is there a way to add the -Wno-overloaded-virtual, flag to
>>> guicast/Makefile instead?  My dumb try failed.
>>>
>>
>>
>> you can try to add it directly to configure.ac like other lines near
>> start of file adding to cfg_cflags.
>>
>
>
> of course exactly because I ignored my own advice and put AX check before
> -Wall checks it was not effective.
>
> Moved those 3 lines below line adding -Wall in configure.ac and now I see
> no clang warnings about overloaded-virtual.
>
>
> still, where is file from
>
>
> http://git.savannah.gnu.org/gitweb/?p=autoconf-archive.git;a=blob_plain;f=m4/ax_check_compile_flag.m4;hb=HEAD
>
> it supposed to be in autoconf-archive but may be we should add it manually
> ? As new file for our tree, then add line to ./autogen.sh copying it to
> newly-created m4 directory ....
>
>
>
>
>
>>
>>> On Tue, Nov 26, 2024 at 6:30 PM Andrew Randrianasulu <
>>> randrianasulu at gmail.com> wrote:
>>>
>>>>
>>>>
>>>> ср, 27 нояб. 2024 г., 04:25 Phyllis Smith <phylsmith2017 at gmail.com>:
>>>>
>>>>> Thanks for passing this along as I have had no success with "using"
>>>>> because I do not know how to use it.  I am hoping that make it less
>>>>> aggressive so that these 526 warnings all go away.
>>>>> Attached patch not working here either (Fedora 40 / C++14.2.1 I
>>>>> think).  From build log file:
>>>>>
>>>>> ./configure: line 17331: syntax error near unexpected token
>>>>> `-Wno-overloaded-virtual,'
>>>>> ./configure: line 17331:
>>>>> `AX_APPEND_COMPILE_FLAGS(-Wno-overloaded-virtual, CFG_CFLAGS)'
>>>>> make: *** No targets specified and no makefile found.  Stop.
>>>>>
>>>>>
>>>>> Will look at some more tomorrow.
>>>>>
>>>>
>>>> be sure to install autoconf-archive (strange that it does not blow up
>>>> in libsndfile that uses similar construct.. mat be they supply new enough
>>>> configure so autoreconf there is not needed ....)
>>>>
>>>>
>>>>
>>>>
>>>>> On Tue, Nov 26, 2024 at 4:04 PM Andrew Randrianasulu <
>>>>> randrianasulu at gmail.com> wrote:
>>>>>
>>>>>>
>>>>>>
>>>>>> сб, 23 нояб. 2024 г., 10:11 Andrew Randrianasulu <
>>>>>> randrianasulu at gmail.com>:
>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> сб, 23 нояб. 2024 г., 02:19 Phyllis Smith <phylsmith2017 at gmail.com>:
>>>>>>>
>>>>>>>> Andrew, the 3 patches have been reviewed as much as I can do and
>>>>>>>> are fine on Linux so I will check them into GIT when I power up the desktop
>>>>>>>> (hopefully tomorrow).  The new 526 Warning errors are actually due to the
>>>>>>>> Fedora 40 C++ compiler.  Right below is the whole message that I did not
>>>>>>>> provide earlier.
>>>>>>>>
>>>>>>>> bcwindowbase.h:222:22: warning: ‘virtual void
>>>>>>>> BC_WindowBase::create_objects()’ was hidden [-Woverloaded-virtual=]
>>>>>>>>   222 |         virtual void create_objects() { return; };
>>>>>>>>       |                      ^~~~~~~~~~~~~~
>>>>>>>> In file included from guicast.h:62:
>>>>>>>> errorbox.h:35:14: note:   by ‘void ErrorBox::create_objects(const
>>>>>>>> char*)’
>>>>>>>>    35 |         void create_objects(const char *text);
>>>>>>>>       |              ^~~~~~~~~~~~~~
>>>>>>>> In file included from bcsubwindow.h:26,
>>>>>>>>                  from bcbutton.h:26,
>>>>>>>>                  from bcprogressbox.C:22:
>>>>>>>>
>>>>>>>>
>>>>>>>> How to fix it is explained at the following url and I will test
>>>>>>>> that but if  i have problems, will let you know!
>>>>>>>>
>>>>>>>>
>>>>>>>> https://stackoverflow.com/questions/15295317/xcode-why-is-a-warning-of-is-hidden-given-with-overloaded-virtual-functions
>>>>>>>>
>>>>>>>>
>>>>>>>
>>>>>>> I also hope it will work! Thanks!
>>>>>>>
>>>>>>
>>>>>> actually, may be warning was a bit too aggressive?
>>>>>>
>>>>>> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109740
>>>>>>
>>>>>> I tried to disable it like libsndfile enables it
>>>>>> but for some reason attached patch not worked with clang?
>>>>>>
>>>>>>
>>>>>>
>>>>>>
>>>>>>
>>>>>>
>>>>>>
>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>>>>>>> On Sun, Nov 17, 2024 at 5:30 PM Andrew Randrianasulu <
>>>>>>>> randrianasulu at gmail.com> wrote:
>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> On Mon, Nov 18, 2024 at 3:11 AM Phyllis Smith <
>>>>>>>>> phylsmith2017 at gmail.com> wrote:
>>>>>>>>>
>>>>>>>>>> As previously mentioned this is a backup laptop where I finally
>>>>>>>>>> installed Fedora 40, which is fedora's latest release.  Tomorrow I will
>>>>>>>>>> boot my regular laptop and build there.  Python version on Fedora 40 is
>>>>>>>>>> 3.12.7.
>>>>>>>>>>
>>>>>>>>>
>>>>>>>>> Mmm, while I dislike new, improved breakage from all-new
>>>>>>>>> into-your-face software ideology ,  what Fedora does usually filters down
>>>>>>>>> to other distros, so we better to know exact error.
>>>>>>>>>
>>>>>>>>> Take your time ...
>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> On Sun, Nov 17, 2024 at 5:04 PM Andrew Randrianasulu <
>>>>>>>>>> randrianasulu at gmail.com> wrote:
>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>> пн, 18 нояб. 2024 г., 02:43 Phyllis Smith <
>>>>>>>>>>> phylsmith2017 at gmail.com>:
>>>>>>>>>>>
>>>>>>>>>>>> Andrew, building on my backup laptop (instead of my standard),
>>>>>>>>>>>> I have 525 new error messages in the log file like:
>>>>>>>>>>>>
>>>>>>>>>>>> /mnt0_new/cin5/cinelerra-5.1/plugins/../guicast/errorbox.h:35:14:
>>>>>>>>>>>> note:   by ‘void ErrorBox::create_objects(const char*)’
>>>>>>>>>>>>
>>>>>>>>>>>> This is quite concerning but I have not actually reviewed the
>>>>>>>>>>>> 3 patches yet, but assume it is the guicast one.
>>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>> patch for guicast/Makefile was adding python3.12 (something
>>>>>>>>>>> pkgin added as dependency while I fetched usual suspects like inkscape,
>>>>>>>>>>> gimp, okular ...)
>>>>>>>>>>>
>>>>>>>>>>> What OS/compiler you have on this laptop?
>>>>>>>>>>>
>>>>>>>>>>> line 35 in guicast/errorbox.h is just
>>>>>>>>>>>
>>>>>>>>>>> void create_objects(const char *text);
>>>>>>>>>>>
>>>>>>>>>>> unchanged by me ....
>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>>>
>>>>>>>>>>>> On Sun, Nov 17, 2024 at 9:42 AM Andrew Randrianasulu <
>>>>>>>>>>>> randrianasulu at gmail.com> wrote:
>>>>>>>>>>>>
>>>>>>>>>>>>> Currently from that system
>>>>>>>>>>>>>
>>>>>>>>>>>>> I still need to set
>>>>>>>>>>>>>
>>>>>>>>>>>>>  export LD_LIBRARY_PATH="/usr/pkg/lib/ffmpeg6"
>>>>>>>>>>>>>
>>>>>>>>>>>>> and set audio output to oss - > /dev/audio but it seems to
>>>>>>>>>>>>> work.
>>>>>>>>>>>>>
>>>>>>>>>>>>> Can you check them by building on Linux? :)
>>>>>>>>>>>>>
>>>>>>>>>>>>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.cinelerra-gg.org/pipermail/cin/attachments/20241216/3ed93673/attachment-0001.htm>