Packages could be linted
illumos-gate packages could be linted with `pkglint`.
A recent commit to gate introduced a new package which could not be installed on IPS distributions due to duplicated metadata.
It would be good to catch things like this earlier by running `pkglint` on the package set as part of the packaging checks.
Updated by Andy Fiddaman 2 months ago
Tested with many full RTI builds on OmniOS r151034 and latest OpenIndiana.
Built packages with and without this change and compared the manifests for all packages. In addition to the lines I explicitly added,
driver/xvm/pv gained a new dependency on
system/kernel/platform - this is because of the updated pkgdepend path for this component.
I introduced failures into manifests and confirmed that they broke the build and were properly picked up by either a failed `pkgdepend` run or by the final `pkglint`.
During testing I found a bug in the 'pkgdepend' command provided by IPS in
both OmniOS and OpenIndiana that occasionally (about 1 run in 20) caused packages to end up with
inconsistent variant information in them and then meant that they (rightly)
failed the pkglint introduced here.
The bug is now fixed in all supported OmniOS releases ('030, '032, '034) and
also in OpenIndiana - the fix is https://github.com/omniosorg/pkg5/commit/f0aceb9b3b918c3d92c6a8efe2d391b236c3e593
Updated by Electric Monk about 1 month ago
- Status changed from New to Closed
- % Done changed from 0 to 100
commit 069e6b7e31ba5dcbc5441b98af272714d9a5455c Author: Andy Fiddaman <firstname.lastname@example.org> Date: 2020-06-02T22:47:31.000Z 12708 Packages could be linted 12717 Stop ignoring pkgdepend errors Reviewed by: Rich Lowe <email@example.com> Reviewed by: John Levon <firstname.lastname@example.org> Reviewed by: Toomas Soome <email@example.com> Approved by: Dan McDonald <firstname.lastname@example.org>