Details
-
Bug
-
Resolution: Fixed
-
P2: Important
-
6.8, 6.9
-
None
-
236954364 (dev), 1c4c53730 (6.8)
Description
The p (documents path) parameter of qmlls does not take a value on 6.8. It's being treated as a flag when QCommandLineOption ctor has an empty default value.
From the other hand, this issue has been fixed on 6.9 with:
https://codereview.qt-project.org/c/qt/qtdeclarative/+/603035
by adding a valueName and also changing p to d as a more meaningful letter.
To keep 6.9 version compatible with 6.8 and also keep them source compatible we can:
- Fix the problem on 6.8 as it's an LTS ( by adding a valueName to QCLO but keeping the option as p)
- Add "p" to the same QCLO as "d" on 6.9, making it backward compatible with 6.8. (https://doc.qt.io/qt-6/qcommandlineoption.html#QCommandLineOption-4)
Attachments
For Gerrit Dashboard: QTBUG-131807 | ||||||
---|---|---|---|---|---|---|
# | Subject | Branch | Project | Status | CR | V |
608882,2 | Fix qmlls document directorty -p cli parameter | dev | qt/qtdeclarative | Status: ABANDONED | 0 | 0 |
609634,2 | qmlls: fix compatibility issue due to removal of "p" flag | dev | qt/qtdeclarative | Status: MERGED | +2 | 0 |
609872,2 | qmlls: fix compatibility issue due to removal of "p" flag | 6.8 | qt/qtdeclarative | Status: MERGED | +2 | 0 |