Powerline: Melanggar perubahan dalam python-i3ipc merusak daftar i3

Dibuat pada 17 Nov 2019  ·  5Komentar  ·  Sumber: powerline/powerline

Pembaruan besar yang relatif baru di i3ipc ( v2.0.1 ) memecah beberapa segmen dan daftar kami di sini.
Saya menemukannya hari ini saat memperbarui semua paket python saya (untuk python 3.8).

Intinya, pembungkus kelas python di sekitar respons JSON i3ipc yang ditangani kode kita telah berubah, sehingga reply['attr'] tidak lagi valid, dan reply.attr atau getattr(reply, 'attr') harus digunakan.
Ini relevan di powerline/listers/i3wm.py dan powerline/segments/i3wm.py.

segment cwm bug

Komentar yang paling membantu

@PH111P ya saya sudah membuat daftar bekerja lagi secara lokal, tapi saya akan menunggu tanggapan di masalah lain sebelum mengubah apa pun di sini

Semua 5 komentar

Tambalan untuk ini akan sangat mudah dibuat, tetapi saya tidak yakin apakah dan bagaimana kompatibilitas ke belakang harus dipertahankan.

Jika saya menafsirkan perbedaan ini dengan benar, maka menggunakan atribut seharusnya sudah berfungsi dengan versi i3ipc yang lebih lama. Melihat bahwa itu digunakan untuk mendukung kedua cara mengakses data, mungkin ini harus diusulkan sebagai perbaikan (bahkan lebih sederhana) di basis kode i3ipc?

Masalah terkait diajukan sebagai altdesktop/i3ipc-python#136

Mengganti reply['attr'] dengan reply.attr memang harus memperbaiki ini.

@PH111P ya saya sudah membuat daftar bekerja lagi secara lokal, tapi saya akan menunggu tanggapan di masalah lain sebelum mengubah apa pun di sini

Sepertinya hulu lebih memilih untuk memecahkan ini hanya karena; Saya kira sudah waktunya untuk patch itu.

Apakah halaman ini membantu?
0 / 5 - 0 peringkat