[yoshimi] Re: V 1.5.7

  • From: Will Godfrey <willgodfrey@xxxxxxxxxxxxxxx>
  • To: yoshimi@xxxxxxxxxxxxx
  • Date: Sat, 10 Mar 2018 08:30:03 +0000

On Sat, 10 Mar 2018 15:13:14 +0800
blubee blubeeme <gurenchan@xxxxxxxxx> wrote:

On Sat, Mar 10, 2018 at 2:55 PM, Ichthyostega <prg@xxxxxxxxxxxxxxx> wrote:

On 10.03.2018 02:39, blubee blubeeme wrote:  
I check for those flags but I didn't see anything out of place. I re-ran  
the  
build and I'm getting this error:
c++: warning: -ljack: 'linker' input unused [-Wunused-command-line-  
argument]  
c++: warning: argument unused during compilation: '-L/usr/local/lib'
[-Wunused-command-line-argument]
/usr/ports/audio/yoshimi/work/yoshimi-bbcb8a2/src/Misc/  
MiscFuncs.cpp:37:10:  
fatal error: 'copyfile.h' file not found
#include <copyfile.h>
         ^~~~~~~~~~~~
1 error generated.
ninja: build stopped: subcommand failed.
*** Error code 1  

Maybe we are looking into the wrong direction.
In such a situation, it is important to find out what *really* stopped the
build. In the case above, how is it possible that the compiler did not
find the header <copyfile.h>  ?
This looks like a system or library header, which means, some of the -I
flags passed to the compiler is not correct and does not point to the
library providing copyfile.h


Incidentally (and this has nothing to do directly with the problem hier),
I noticed the following waring in the Pastbin-Log:

 
In file included from  
/usr/ports/audio/yoshimi/work/yoshimi-bbcb8a2/src/UI/WidgetPDial.h:34:  
/usr/ports/audio/yoshimi/work/yoshimi-bbcb8a2/src/UI/  
DynamicTooltip.h:52:8:  
warning: 'DynTooltip::show' hides overloaded virtual function  
[-Woverloaded-virtual]  
 void show(float timeout=0);
      ^
/usr/local/include/FL/Fl_Menu_Window.H:35:8: note: hidden overloaded  
virtual  
function 'Fl_Menu_Window::show' declared here: different number of  
parameters
(0 vs 1)  
 void show();
      ^  


IMHO this kind of warning should be taken seriously and investigated.
This kind of errors can be quite insidious, because as a result the
wrong overload of a method gets called, which typically does not break
the program, but could cause subtly wrong behavior.

-- Hermann


Yoshimi source code is available from either: https://sourceforge.net/
projects/yoshimi
Or: https://github.com/Yoshimi/yoshimi
Our list archive is at: https://www.freelists.org/archive/yoshimi
To post, email to yoshimi@xxxxxxxxxxxxx
 
Looking into this a bit more seems these errors popped up around 9 days ago
when the copyfile functions were added.

There's no copyfile.h in FreeBSD as far as I can see, what's available to
use in it's place.

Best

This was *exactly* my concern. I've seen conflicting information on-line, but
the consensus seemed to be that freebsd couldn't handle sendfile locally as it
works via sockets (linux does both).
If copyfile also is not available case then we are going to have to write our
own file copy routine, which I really wanted to avoid. Currently, this is only
needed by the new bank import/export feature.

Just to prove the point can you look in /src/Misc/MiscFuncs.cpp
Cooment out the following lines:
36 - 38
40
356 - 358
363

Compile, and see if those features actually work! I'm expecting them to fail :(

Thanks for sticking with this - much appreciated.

It might be possible to use the save and load text routines (later in MiscFuncs)
but they will be quite slow.

Thanks for sticking with this - much appreciated.

Will.

-- 
Will J Godfrey
http://www.musically.me.uk
Say you have a poem and I have a tune.
Exchange them and we can both have a poem, a tune, and a song.
Yoshimi source code is available from either: 
https://sourceforge.net/projects/yoshimi
Or: https://github.com/Yoshimi/yoshimi
Our list archive is at: https://www.freelists.org/archive/yoshimi
To post, email to yoshimi@xxxxxxxxxxxxx

Other related posts:

  1. Home
  2. yoshimi
  3. Archive
  4. 03-2018