Manifest file with invalid syntax reported as missing
Bug #1409815 reported by
dobey
This bug affects 2 people
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
qtcreator-plugin-ubuntu (Ubuntu) |
Fix Released
|
High
|
Benjamin Zeller |
Bug Description
Why trying to build a click package in the SDK, if the manifest.json file exists in the target directory, but has invalid syntax such as a missing ending brace, it is reported as missing, rather than having a syntax error. This leads one to look for some reason why the SDK would think the file is missing despite it being in the directory, such as permissions or other issues that might arise from installation rules in the build system, rather than looking at the contents of the file itself to see they are wrong.
A missing file, and syntax errors in the file, should be reported differently, as they are very different and distinct problems.
Related branches
lp:~zeller-benjamin/qtcreator-plugin-ubuntu/lp1409815
- PS Jenkins bot: Approve (continuous-integration)
- Zoltan Balogh: Approve
-
Diff: 281 lines (+82/-18)11 files modifiedsrc/ubuntu/ubuntuabstractguieditordocument.cpp (+5/-1)
src/ubuntu/ubuntuabstractguieditorwidget.cpp (+5/-0)
src/ubuntu/ubuntuabstractguieditorwidget.h (+2/-0)
src/ubuntu/ubuntuclickmanifest.cpp (+11/-4)
src/ubuntu/ubuntuclickmanifest.h (+1/-1)
src/ubuntu/ubuntulocalrunconfiguration.cpp (+3/-2)
src/ubuntu/ubuntulocalrunconfigurationfactory.cpp (+11/-3)
src/ubuntu/ubuntumanifesteditorwidget.cpp (+15/-1)
src/ubuntu/ubuntumanifesteditorwidget.h (+1/-0)
src/ubuntu/ubuntupackagestep.cpp (+19/-4)
src/ubuntu/ubunturemoterunconfiguration.cpp (+9/-2)
To post a comment you must log in.
Status changed to 'Confirmed' because the bug affects multiple users.