On Thu, Jan 22, 2026 at 10:51 PM Seyed Mohamad Amin Modaresi <[email protected]> wrote: > Is there any sponsor for gxml? > I'll keep it under team. > > https://salsa.debian.org/amin_modaresi/gxml
Thank you for your persistence in working to get this package into Debian. I believe the package is almost ready for upload now. I needed to make some changes first. 1. The instructions at https://wiki.debian.org/Gnome/Git#Initial_packaging weren't followed to provide the full upstream git history, so I rebuilt the repo. 2. Later, I realized there was a copyright violation so I reported it upstream [1]. I then needed to add these files to Files-Excluded and then rebuilt the repo again. (I haven't written documentation for how to exclude files for an initial import.) For Debian's purposes, this issue is now fixed and won't block us from uploading. 3. I renamed the source package to gxml instead of the versioned name you picked. We prefer unversioned source package names unless we are forced to have more than one version of a project in Debian. Even then, we generally prefer to use an unversioned name for the newer, recommended source package. (For instance, when we packaged the new version of uhttpmock, we repackaged the old version as uhttpmock0. Later, we were able to remove uhttpmock0 since nothing was using it any more.) 4. I also renamed the -dev packages to remove the version number. This follows the recommendation in the first sentence of Debian Policy § 8.4 [2]. 5. I updated your disable-tests patch to only disable the test that doesn't work (because of libxml2 changes) and the 2 tests I broke by removing the copyrighted files. 6. The new team repo is at https://salsa.debian.org/gnome-team/extras/gxml Required Change ============== 7. You need to add a libgxml-common package for the locales instead of having the -dev package install them. Have the library package depend on this new package. Suggested Changes ================ Other changes can be done after we upload the package to Debian's NEW queue. 8. Your patches say Forwarded: yes but they don't appear to have been forwarded. I think the pkgconf patch is handled by https://gitlab.gnome.org/GNOME/gxml/-/merge_requests/30 but the python3 patch should be forwarded. 9. Please remove Recommends: devhelp. devhelp will be removed from Debian soon and replaced by the new manuals package and it's unnecessary to have packages recommend it. 10. Please remove Priority: optional [3] 11. The build log has these warnings. Please fix them. Ignore the <!nogir> part; most packages can't use that. When it says used but not defined, remove that dependency. When it says unused but defined, add that field. dh_girepository: warning: Missing Build-Depends: gir1.2-gobject-2.0-dev (ideally with <!nogir>) dh_girepository: warning: Missing Build-Depends: gir1.2-gee-0.8-dev (ideally with <!nogir>) dh_girepository: warning: Missing Build-Depends: gir1.2-gio-2.0-dev (ideally with <!nogir>) dh_girepository: warning: Missing Build-Depends: gir1.2-libxml2-2.0-dev (ideally with <!nogir>) dpkg-gencontrol: warning: package libgxml-dev: substitution variable ${gir:Provides} unused, but is defined dpkg-gencontrol: warning: Depends field of package gir1.2-gxml-0.20: substitution variable ${shlibs:Depends} used, but is not defined dpkg-gencontrol: warning: Depends field of package libgxml-doc: substitution variable ${shlibs:Depends} used, but is not defined 12. You can remove the direct Depends libgxml-dev -> gir1.2-gxml-0.20 since that's handled for us by ${gir:Depends} 13. The only dependencies that libgxml-0.20-2.0.2 should have are ${misc:Depends}, ${shlibs:Depends} The only depends that gir1.2-gxml-0.20 should have are ${gir:Depends}, ${misc:Depends} 14. Please remove debian/tmp/ from the *.install files. It makes those files easier to read. 15. For the gir package, it's better to use -usr/lib/*-linux-gnu/girepository-*/ +usr/lib/*/girepository-1.0 The style you used won't work for hurd. 16. Set debian/tests/test1.sh as executable. (It still works without that, but it's best practice for scripts to be executable.) 17. I recommend removing debian/tests/example.vala and updating test1.sh to use "examples/vala" instead of "debian/tests" 18. Lintian warns that example.vala doesn't need to be executable. Please forward that suggestion upstream. 19. The -dev package description says that the package contains the static library. It doesn't (and that's ok) so please remove that part of the description. [1] https://gitlab.gnome.org/GNOME/gxml/-/issues/54 [2] https://www.debian.org/doc/debian-policy/ch-sharedlibs.html#development-files [3] https://www.debian.org/doc/debian-policy/upgrading-checklist.html#version-4-7-3 Jeremy Bícha
