| *** tristan <tristan!tristan@1.232.31.153> has joined #buildstream | 05:28 | |
| *** ChanServ sets mode: +o tristan | 05:28 | |
| *** tpollard <tpollard!tompollard@167.98.27.226> has joined #buildstream | 08:17 | |
| *** tristan <tristan!tristan@1.232.31.153> has quit IRC | 09:25 | |
| *** tristan <tristan!tristan@1.232.31.153> has joined #buildstream | 09:26 | |
| *** ChanServ sets mode: +o tristan | 09:26 | |
| *** CTtpollard <CTtpollard!tompollard@167.98.27.226> has joined #buildstream | 09:31 | |
| *** tpollard <tpollard!tompollard@167.98.27.226> has quit IRC | 09:32 | |
| *** CTtpollard <CTtpollard!tompollard@167.98.27.226> has quit IRC | 09:48 | |
| *** tpollard <tpollard!tompollard@167.98.27.226> has joined #buildstream | 09:49 | |
| *** tristan <tristan!tristan@1.232.31.153> has quit IRC | 09:55 | |
| *** CTtpollard <CTtpollard!tompollard@167.98.27.226> has joined #buildstream | 12:57 | |
| *** tpollard <tpollard!tompollard@167.98.27.226> has quit IRC | 12:58 | |
| *** tpollard <tpollard!tompollard@167.98.27.226> has joined #buildstream | 13:25 | |
| *** CTtpollard <CTtpollard!tompollard@167.98.27.226> has quit IRC | 13:25 | |
| *** SamThursfield[m] <SamThursfield[m]!ssssammatr@2001:470:1af1:104:0:0:0:220c> has joined #buildstream | 13:52 | |
| SamThursfield[m] | hello, i had a quick look at adding python3.11 support to Bst, and first noticed that the docs on how to do this aren't merged yet | 13:52 |
|---|---|---|
| SamThursfield[m] | https://github.com/apache/buildstream/pull/1738/ - does anyone fancy reviewing ? | 13:52 |
| *** nanonyme <nanonyme!nanonyme@2001:470:1af1:104:0:0:0:45ea> has joined #buildstream | 13:53 | |
| nanonyme | Looks good to me. It's not currently tested on Python 3.11 even, I guess you're going to start with that? | 13:53 |
| nanonyme | Fedora 37 should be added to be tested, that will cover Python 3.11 | 13:54 |
| SamThursfield[m] | thanks, yeah the doc starts with updating tox.ini to include cp311 in the testing | 14:04 |
| SamThursfield[m] | now hitting the ruamel.yaml install error caused by them using old Cython version. I'll try regenerating the requirements.txt to pull in latest ruamel.yaml | 14:04 |
| SamThursfield[m] | now same issue for pyroaring | 14:11 |
| nanonyme | Yay | 14:11 |
| SamThursfield[m] | but I don't actually see an update to ruamel.yaml either, https://pypi.org/project/ruamel.yaml/#history shows the last release as Feb 2022 | 14:12 |
| nanonyme | The bug was in ruamel.yaml.clib, not ruamel.yaml. | 14:13 |
| SamThursfield[m] | aha | 14:13 |
| SamThursfield[m] | not much action in upstream pyroaring about this, https://github.com/Ezibenroc/PyRoaringBitMap/issues/78 has been opened just yesterday and looks like a big chunk of work | 14:16 |
| nanonyme | I think both projects are doing it wrong. They should not be shipping C code but just require whoever builds the sdist to use Cython | 14:17 |
| SamThursfield[m] | agreed | 14:18 |
| nanonyme | ruamel.yaml.clib mostly only seems to be releasing because Python C API becomes incompatible with generated C code | 14:19 |
| nanonyme | Sam Thursfield: I guess you could regenerate the sources with newer Cython (same as what was used for fixing ruamel.yaml.clib probably), you could make a PR and see if they accept | 14:35 |
| nanonyme | The code is supposedly compatible with older Python versions too | 14:37 |
| *** tpollard <tpollard!tompollard@167.98.27.226> has quit IRC | 15:57 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!