Re: [Denemo-devel] The lilypond build system adopted to build Denemo for various OS

[Richard Shann]

I forgot to mention that the actions/editscripts is also working in -6
properly - you can see this when you try to edit a SmallerStaff
directive - it only offers to delete it, because that is the only option
in the editscripts/SmallerStaff script.
The only thing not working AFAIK is finding the bitmaps - this must be
some internal bug in denemo, surely...
Richard

On Tue, 2009-07-07 at 17:19 +0100, Richard Shann wrote:
> On Tue, 2009-07-07 at 16:46 +0200, Jan Nieuwenhuizen wrote:
> > On di, 2009-07-07 at 15:05 +0100, Richard Shann wrote:
> > 
> > > > Have a look at -5 from
> > > > http://lilypond.org/blog/janneke/software/denemo
> > > > 
> > > I've installed this on a windows box, and can report progress:
> > 
> > Ah, great. 
> > >   The actions/menus, bitmaps, editscripts are all installed.
> > > However the program  fails to find the Denemo.ttf font, and in fact
> > > there are other font not found messages, which IIRC I get with my
> > > hand-built version as well, though mine does find Denemo.ttf, though
> > > where it gets it from I don't know.
> > 
> > The fonts problem is fixed in -6.
> I have just got back from testing -6 and can report that indeed this is
> solved.
> Moreover, the installation of the scripts is mostly solved, the
> actions/menus/... are working, but denemo fails to find the
> actions/bitmaps files when the scripts are run.
> I have looked thru the code where it is trying to find the bitmaps in
> denemo/src/view.c which looks ok
> 
>     if(!g_file_test(filename, G_FILE_TEST_EXISTS)) {
>       g_free(filename);
>       filename = g_build_filename (get_data_dir (), "actions", "bitmaps", 
> pngname, 
>                                  NULL);
> ...
> 
> So this should create
>  .../usr/share/denemo/actions/bitmaps/RepeatStartEnd.png
> or some such - but in winspeak - and that exists.
> 
> (I did find a bug -  the filename on entry to this fragment is set by
> this:
>  
> filename = g_build_filename (get_data_dir (), "download", "actions", 
> "bitmaps", pngname, 
>                                NULL);
> which would never exist - the download directory is never in
> system-wide /usr/..., it is in ~/.denemo/actions when you have updated
> from denemo.org. However this is benign here :-)
> 
> Tantalizingly close...
> Richard
> 
> > 
> > > More serious though is a failure to find the installed actions/menus etc
> > > at runtime. The relevant bit of code in denemo/src/utils.c is as
> > > follows:
> > > 
> > > #ifdef G_OS_WIN32
> > >     gchar *rootdir = g_win32_get_package_installation_directory (NULL, 
> > > NULL);
> > >     datadir = g_build_filename (rootdir, "share", "denemo", NULL);
> > >     g_free (rootdir);
> > > #else /* not G_OS_WIN32 */
> > > 
> > > I guess this should be
> > >     
> > > datadir = g_build_filename (rootdir, "usr", "share", "denemo", NULL);
> > > 
> > > which I will change.
> > 
> > That won't help.  In fact, I removed usr, because we had /usr/usr.
> > I just added these g_prints in utils.c, and here's what
> > wine says
> > 
> >     gchar *rootdir = g_win32_get_package_installation_directory (NULL,
> > NULL);
> >     datadir = g_build_filename (rootdir, "share", "denemo", NULL);
> >     g_print ("rootdir=%s\n", rootdir);
> >     g_print ("datadir=%s\n", datadir);
> > 
> > gives
> > 
> >     rootdir=C:\Programma Bestanden\GNU_Denemo\usr
> >     datadir=C:\Programma Bestanden\GNU_Denemo\usr\share\denemo
> > 
> > So, it must be something else?  I added the prints to latest
> > GUB, but there's no -7 yet to test it.  Maybe you can try to
> > build it...
> > 
> > Greetings,
> > Jan.
> > 
> 
> 
> 
> _______________________________________________
> Denemo-devel mailing list
> address@hidden
> http://lists.gnu.org/mailman/listinfo/denemo-devel