bugDenemo - Bugs: bug #46428, LilyPond import is broken

 
 

bug #46428: LilyPond import is broken

Submitter:  Richard Shann <rshann>
Submitted:  Thu 12 Nov 2015 08:49:31 AM UTC
   
 
Category:  None Severity:  3 - Normal
Item Group:  None Status:  Postponed
Privacy:  Public Assigned to:  None
Originator Name:  Open/Closed:  Open
* Mandatory Fields

Add a New Comment Rich Markup
   

Sat 14 Nov 2015 04:01:02 PM UTC, comment #3: 

The import is only done with all work saved.
now in 2.0.1

The longer term thing would be to examine the possibility of defining #{ etc in the scheme environment which the imput parser is using.... the "etc" in this case might involve a lot of definitions that LilyPond provides, without which there will be no way of swallowing the correct amount of input after the # token is found.

Richard Shann <rshann>
Group administrator
Sat 14 Nov 2015 02:33:29 PM UTC, comment #2: 

I've pushed a fix that means that ordinary LilyPond output from Denemo avoids that problematic syntax.
It would be good to add a check that everything was saved before risking the parser hanging (unless the problem could be solved in the parser ... difficult to see how - what does Frescobaldi do to parse over embedded scheme?)

Richard Shann <rshann>
Group administrator
Sat 14 Nov 2015 12:41:03 PM UTC, comment #1: 

Not all LilyPond files are failing - Denemo's own ones are and it seems to be this syntax:

DenemoGlobalTranspose = #(define-music-function (parser location arg)(ly:music?) #{\transpose c c #arg #})


That is causing the parsing to hang - my guess is that the first # causes the parser to ask the guile expression reader to exhaust the stuff that follows, but that stuff contains the #{ #} syntax which is not Scheme but special LilyPond syntax. So this then causes guile to throw an error and no recovery appears to be made from it.

Richard Shann <rshann>
Group administrator
Thu 12 Nov 2015 08:49:31 AM UTC, original submission:  

It seems to lock in a thread wait.
This seems to be true for several versions past. currently 2.0.1

Richard Shann <rshann>
Group administrator

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by rshann (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 2 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2015-11-14 rshann StatusIn Progress Postponed
    2015-11-14 rshann StatusNone In Progress

    Back to the top

    Powered by Savane 3.13-f8d8.
    Corresponding source code