Well no, my push has been rejected, so here are the updated hashes:
~~~
commit c114c0444b62d3b3c8a12b0124f807218eeee859
Author: Jonas Hahnfeld <[email protected]>
Date:   Sat Nov 23 10:03:14 2019 +0100

    Issue 5617/3: Open midi and musicxml files in binary mode
    
    When reading input files, only decode once we are sure that the
    content is not compressed and is properly encoded with UTF-8.

commit 92fe023174739455feb4c645e9aa2565a058dc1e
Author: Jonas Hahnfeld <[email protected]>
Date:   Sat Nov 23 09:57:50 2019 +0100

    Issue 5617/2: Submit bytes-like objects to processes
    
    Encode strings before writing to input and decode result after reading
    from output or error stream.

commit 2dc93ea87a90ca5ccbbc85a9d4e01fc59ce3ee14
Author: Jonas Hahnfeld <[email protected]>
Date:   Sat Nov 23 09:48:27 2019 +0100

    Issue 5617/1: Encode strings before hashing
    
    Python 3 requires "bytes-like objects". Apparently we can already
    do so in Python 2, even though not needed for correctness.
~~~


---

** [issues:#5617] Prepare for encoding changes in Python 3**

**Status:** Fixed
**Labels:** Fixed_2_21_0 
**Created:** Sat Nov 23, 2019 03:08 PM UTC by Jonas Hahnfeld
**Last Updated:** Thu Nov 28, 2019 07:28 PM UTC
**Owner:** Jonas Hahnfeld


Prepare for encoding changes in Python 3

This is most of the remaining diff for porting to Python 3,
but this also works in Python 2.4 already (even though not
strictly needed).

Individual changes:
1. Encode strings before hashing

Python 3 requires "bytes-like objects". Apparently we can already
do so in Python 2, even though not needed for correctness.

2. Submit bytes-like objects to processes

Encode strings before writing to input and decode result after reading
from output or error stream.

3. Open midi and musicxml files in binary mode

Only decode once we are sure that the content is not compressed.

http://codereview.appspot.com/573280043


---

Sent from sourceforge.net because [email protected] 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
[email protected]
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto
  • ... Auto mailings of changes to Lily Issues via Testlilyissues-auto via Automated messages for lilypond development
    • ... Auto mailings of changes to Lily Issues via Testlilyissues-auto via Automated messages for lilypond development
    • ... Auto mailings of changes to Lily Issues via Testlilyissues-auto via Automated messages for lilypond development
    • ... Auto mailings of changes to Lily Issues via Testlilyissues-auto via Automated messages for lilypond development
    • ... Auto mailings of changes to Lily Issues via Testlilyissues-auto via Automated messages for lilypond development
    • ... Auto mailings of changes to Lily Issues via Testlilyissues-auto via Automated messages for lilypond development

Reply via email to