Fix: LXQt sürüm tespiti çalışmıyor (get_desktop_env_version) - #57
Open
muhammetali wants to merge 1 commit into
Open
Fix: LXQt sürüm tespiti çalışmıyor (get_desktop_env_version)#57muhammetali wants to merge 1 commit into
muhammetali wants to merge 1 commit into
Conversation
get_desktop_env_version fonksiyonunun lxqt dalı, lxqt-about --version
çıktısını satırlara ayırmadan dolaşıyor. Döngü dize üzerinde karakter
karakter ilerlediği için "liblxqt" araması hiçbir zaman eşleşmiyor ve
sürüm her zaman boş kalıyor.
Satırlarda dönen diğer iki dal (xfce ve gnome) output.split("\n")
kullanıyor, lxqt dalında bu atlanmış.
Pardus 23.0 üzerinde depodan kurulu pardus-software 0.8.3 ile doğrulandı:
$ lxqt-about --version
lxqt-about 1.2.0
liblxqt 1.2.0
Qt 5.15.6
get_desktop_env_version("LXQt") önce: '' sonra: '1.2.0'
Sunucuya gönderilen distro bilgisi de düzeliyor:
önce : Pardus GNU/Linux, 23.0, yirmiuc, (LXQt , X11)
sonra: Pardus GNU/Linux, 23.0, yirmiuc, (LXQt 1.2.0, X11)
Aynı makinede yama öncesi/sonrası karşılaştırıldığında xfce, gnome, kde,
mate, cinnamon ve budgie dallarının çıktısı değişmiyor.
muhammetali
force-pushed
the
fix/lxqt-version-detection
branch
from
September 9, 2026 16:10
2c516d4 to
788e06d
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Utils.get_desktop_env_version()fonksiyonununlxqtdalı,lxqt-about --versionçıktısını satırlara ayırmadan dolaşıyor:
Döngü bir dize üzerinde döndüğü için
lineher adımda tek bir karakter oluyor ve"liblxqt" in linekoşulu hiçbir zaman sağlanmıyor. Sonuç olarak LXQt oturumlarındasürüm bilgisi her zaman boş kalıyor.
Fonksiyonda satırlarda dönen diğer iki dal (
xfcevegnome)output.split("\n")kullanıyor;
lxqtdalında busplitatlanmış. (cinnamon,matevekdedallarısatırlara hiç ayırmıyor, tek satırlık çıktıyı doğrudan ayrıştırıyor.)
Doğrulama
Pardus 23.0 (yirmiuc) üzerinde, depodan kurulu
pardus-software 0.8.3ile(
/usr/share/pardus/pardus-software/src/Utils.py), git kopyası ile değil:liblxqtsatırı çıktıda mevcut ve sürüm okunabilir durumda. Buna rağmen:Yama sonrası aynı çağrı
'1.2.0'döndürüyor.Etkisi
Sonuç
user_desktop_env→user_distro_fullzinciriyle sunucuya gönderilendistroalanına gidiyor:Pardus GNU/Linux, 23.0, yirmiuc, (LXQt , X11)Pardus GNU/Linux, 23.0, yirmiuc, (LXQt 1.2.0, X11)Mevcut durumda virgülden önce boşluk kalıyor ve sürüm bilgisi kayboluyor.
Regresyon kontrolü
Aynı makinede yama öncesi ve sonrası tüm dallar karşılaştırıldı:
'''1.2.0''4.18''4.18'''''''''''''''''''''Değişiklik tek satır ve yalnızca
lxqtdalını etkiliyor.Not
Aynı hata
pardus-update/src/Utils.pydosyasında da var, ancak orada fonksiyonçağrılmadığı için ayrı bir PR açmadım.