lilypond-auto
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5665 Remove b


From: Auto mailings of changes to Lily Issues via Testlilyissues-auto
Subject: [Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5665 Remove broken and undocumented tracing features
Date: Thu, 23 Jan 2020 17:25:12 -0000

Patch counted down - please push.


[issues:#5665] Remove broken and undocumented tracing features

Status: Started
Created: Sat Jan 18, 2020 05:02 PM UTC by Jonas Hahnfeld
Last Updated: Tue Jan 21, 2020 04:37 PM UTC
Owner: Jonas Hahnfeld

Remove broken and undocumented tracing features

Individual changes:
1. Drop -dtrace-scheme-coverage

As the initial commit dee82b1d0b of scm/coverage.scm notes, this
module requires support for memoize-symbol. This was indeed added
for Guile 1.9.0, but again removed in version 1.9.13. That means
there never was a stable version of Guile that worked with this.

When building / running with current Guile 1.8.8, the executable
crashes because the interpreter raise()s from within a critical
section and (at least my) libc kills the process:
$ guile1.8 -c "(trap-enable 'memoize-symbol)"
throw from within critical section.
Aborted (core dumped)

(With later releases of Guile, there's at least an error message:
$ guile2.0 -c "(trap-enable 'memoize-symbol)"
[...]
ERROR: In procedure memoize-variable-access!:
ERROR: Unbound variable: trap-enable

This is because the mentioned (unstable) release 1.9.13 completely
removed not only memoize-symbol, but the whole trap-* interface.)

  1. Drop -dtrace-memory-frequency

The existing documentation is outdated:
* The option is not bool, but takes an integer. Maybe the frequency,
but in what unit?
* The mentioned patch has been part of Guile since version 1.8.2.
However I can't find the added variable since release 1.9.3, not
unlikely that the module would not work with later versions.

Additionally there is no guide on how to use the produced files (.stacks
and .graph). If somebody has more information and wants this feature back,
please revert this commit and add appropriate documentation.

  1. Delete scripts to measure coverage

build-coverage.sh is meant to configure LilyPond for gathering coverage
information, then runs a full 'make test' and finally calls coverage.py
to postprocess the data.

Apart from using the broken -dtrace-scheme-coverage, build-coverage.sh
only works when called from the repository's root directory (as opposed
to just creating a new build directory). Furthermore it randomly touches
midi2ly and its manpage midi2ly.1 which leads to errors at first.
Finally the called covarage.py doesn't seem to handle coverage data
emitted by the latest GCC, which makes it useless going forward.

  1. Delete script build-profile.sh

It's not described in the documentation and seems to expect files
in input/mutopia which was removed back in 2009.

  1. Update documentation about tracing options

http://codereview.appspot.com/577320043


Sent from sourceforge.net because address@hidden is subscribed to https://sourceforge.net/p/testlilyissues/issues/

To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/testlilyissues/admin/issues/options. Or, if this is a mailing list, you can unsubscribe from the mailing list.

_______________________________________________
Testlilyissues-auto mailing list
address@hidden
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto

reply via email to

[Prev in Thread] Current Thread [Next in Thread]