> With this patch lilypond-invoke-editor only
> handles textedit URIs, and it does no longer
> use the systems command processor but
> guiles system* procedure for those URIs.
AFAIU, this is not completely true. It does handle other URIs. If there's no
intent to fix the command injection
AFAIU, the fix of lilypond-invoke-editor is not merged. I still have this:
~~~
(define (run-browser uri)
(system
(if (getenv "BROWSER")
(format #f "~a ~a" (getenv "BROWSER") uri)
(format #f "firefox -remote 'OpenURL(~a,new-tab)'" uri
~~~
---
** [issues:#5243] Fix
- **Patch**: countdown --> push
- **Comment**:
Patch counted down - Bernhard can you make a git-formatted patch against
current master and attach it, I can then push it for you. James
---
** [issues:#5289] update help2man**
**Status:** Started
**Created:** Mon Mar 12, 2018 04:21 PM UTC by
- **Patch**: countdown --> push
- **Comment**:
Patch counted down - please push
---
** [issues:#5291] Allow event functions from partial functions or partial
markups**
**Status:** Started
**Created:** Wed Mar 14, 2018 03:37 PM UTC by David Kastrup
**Last Updated:** Fri Mar 16, 2018 06:18 PM
- **Patch**: countdown --> push
- **Comment**:
Patch counted down - Bernhard can you make a git-formatted patch against
current master and attach it, I can then push it for you. James
---
** [issues:#5290] Use convert -strip**
**Status:** Started
**Created:** Tue Mar 13, 2018 10:29 AM UTC
files
Attachments:
-
[2018-03-19-153859_REGTEST.png](https://sourceforge.net/p/testlilyissues/issues/_discuss/thread/f70cdd64/a2e4/attachment/2018-03-19-153859_REGTEST.png)
(291.8 kB; image/png)
-
- **Needs**: -->
- **Patch**: new --> review
- **Type**: --> Enhancement
- **Comment**:
Passes make, make check and a full make doc.
Reg test diffs attached - I am also getting a number of programming errors, but
I am not sure if these are those cases where they happen because of the
The fix is merged. It just does not involve run-browser. This would likely
warrant additional shell-quoting here or possibly just removing the run-browser
functionality if we see no clean way to make this work.
---
** [issues:#5243] Fix security problem in lilypond-invoke-editor**
- **labels**: --> Fixed_2_21_0
- **status**: Started --> Fixed
- **Patch**: push -->
- **Comment**:
Pushed to staging as
commit d91fede8a101f8b7208b1cc21e920ad2c4327e38
Author: David Kastrup
Date: Wed Mar 14 15:03:27 2018 +0100
Issue 5291/3: Allow event functions from
> I am also getting a number of programming errors, but I am not sure if these
> are those cases where they happen because of the multithreaded testing I do
I don’t get those programming errors (running both make test-baseline and make
check with -j5 CPU_COUNT=5).
---
** [issues:#5277] add
- Description has changed:
Diff:
- **Needs**: -->
- **Patch**: new --> review
- **Comment**:
Passes make, make check and a full make doc.
---
** [issues:#5293] Changes: add \n for niente**
**Status:** Started
**Created:** Sun Mar 18, 2018 02:15 PM UTC by Malte Meyn
**Last
- Description has changed:
Diff:
- **Needs**: -->
- **Patch**: new --> review
- **Comment**:
Passes make, make check and a full make doc. Full reg test here
https://cloud.indie.host/s/DtKeMaHDk4qTgEP
---
** [issues:#5292] fix width of 8th and 16th flags**
**Status:**
12 matches
Mail list logo