FS#77106 - [gloobus-preview] Please consider switching source from bzr+lp to bzr

Attached to Project: Community Packages
Opened by loqs (loqs) - Thursday, 12 January 2023, 22:03 GMT
Last edited by Toolybird (Toolybird) - Monday, 20 November 2023, 19:43 GMT
Task Type General Gripe
Category Packages
Status Closed
Assigned To Balló György (City-busz)
Architecture All
Severity Very Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description:
bzr+lp requires the unpackaged py-launchpadlib plus a Ubuntu One account plus an authentication token for the initial clone.
I believe this can be avoided by changing the protocol, see attached diff. I did not add python-dulwich to makedepends as that is covered by  FS#76977 .

Additional info:
* gloobus-preview 0.4.5.336-4
* config and/or log files etc.
* link to upstream bug report, if any

Steps to reproduce:
$ git clone https://github.com/archlinux/svntogit-community.git --single-branch --branch 'packages/gloobus-preview' gloobus-preview
$ cd gloobus-preview/trunk/
$ extra-x86_64-build
:: Synchronizing package databases...
core downloading...
extra downloading...
community downloading...
:: Starting full system upgrade...
there is nothing to do
==> Building in chroot for [extra] (x86_64)...
==> Synchronizing chroot copy [/var/lib/archbuild/extra-x86_64/root] -> [builduser]...done
==> Making package: gloobus-preview 0.4.5.336-4 (Thu Jan 12 21:54:57 2023)
==> Retrieving sources...
-> Branching lp:gloobus-preview...
brz: ERROR: launchpadlib is required for Launchpad API access. Please install the launchpadlib package.
==> ERROR: Failure while branching lp:gloobus-preview
Aborting...
==> ERROR: Could not download sources.
This task depends upon

Closed by  Toolybird (Toolybird)
Monday, 20 November 2023, 19:43 GMT
Reason for closing:  Fixed
Additional comments about closing:  gloobus-preview-0.4.5.336-6
Comment by Levi Zim (kxxt) - Sunday, 04 June 2023, 14:04 GMT
Ubuntu One account and Authentication are not required. Installing python-launchpadlib works for me
Comment by loqs (loqs) - Sunday, 04 June 2023, 14:36 GMT
pacman -U python-lazr-uri/python-lazr-uri-1.0.6-1-any.pkg.tar.zst python-wadllib/python-wadllib-1.3.6-1-any.pkg.tar.zst python-lazr-restfulclient/python-lazr-restfulclient-0.14.4-1-any.pkg.tar.zst python-launchpadlib/python-launchpadlib-1.11.0-1-any.pkg.tar.zst
loading packages...
resolving dependencies...
looking for conflicting packages...

Packages (10) python-importlib-metadata-5.0.0-5 python-jaraco.classes-3.2.3-3
python-jeepney-0.8.0-2 python-keyring-23.13.1-1
python-secretstorage-3.3.3-2 python-zipp-3.15.0-2
python-launchpadlib-1.11.0-1 python-lazr-restfulclient-0.14.4-1
python-lazr-uri-1.0.6-1 python-wadllib-1.3.6-1

Total Installed Size: 4.43 MiB
Net Upgrade Size: 3.60 MiB

:: Proceed with installation? [Y/n] y
(10/10) checking keys in keyring [######################] 100%
( 9/10) checking package integrity [######################] 100%
( 9/10) loading package files [######################] 100%
(10/10) checking for file conflicts [######################] 100%
(10/10) checking available disk space [######################] 100%
:: Processing package changes...
( 1/10) reinstalling python-lazr-uri [######################] 100%
( 2/10) reinstalling python-wadllib [######################] 100%
( 3/10) reinstalling python-lazr-restfulclient [######################] 100%
( 4/10) installing python-jaraco.classes [######################] 100%
( 5/10) installing python-jeepney [######################] 100%
( 6/10) installing python-secretstorage [######################] 100%
( 7/10) installing python-zipp [######################] 100%
( 8/10) installing python-importlib-metadata [######################] 100%
( 9/10) installing python-keyring [######################] 100%
Optional dependencies for python-keyring
python-keyrings-alt: Alternative backends
python-dbus: kwallet backend [installed]
(10/10) installing python-launchpadlib [######################] 100%
Optional dependencies for python-launchpadlib
python-setuptools [installed]
python-testresources
:: Running post-transaction hooks...
(1/1) Arming ConditionNeedsUpdate...
$ pkgctl repo clone --protocol=https gloobus-preview
==> Cloning gloobus-preview ...
Cloning into 'gloobus-preview'...
remote: Enumerating objects: 85, done.
remote: Counting objects: 100% (85/85), done.
remote: Compressing objects: 100% (72/72), done.
remote: Total 85 (delta 23), reused 0 (delta 0), pack-reused 0
Receiving objects: 100% (85/85), 14.06 KiB | 14.06 MiB/s, done.
Resolving deltas: 100% (23/23), done.
==> Collecting packager identity from makepkg.conf
-> name : undefined
-> email : undefined
-> gpg-key : undefined
-> protocol: https
==> Configuring gloobus-preview
$ cd gloobus-preview/
$ pkgctl build
==> Updating pacman database cache
:: Synchronizing package databases...
core downloading...
extra downloading...
multilib downloading...
==> Building gloobus-preview
-> repo: extra
-> arch: x86_64
-> worker: builduser-1
==> Building gloobus-preview for [extra] (x86_64)
:: Synchronizing package databases...
core downloading...
extra downloading...
:: Starting full system upgrade...
there is nothing to do
==> Building in chroot for [extra] (x86_64)...
==> Synchronizing chroot copy [/var/lib/archbuild/extra-x86_64/root] -> [builduser-1]...done
==> Making package: gloobus-preview 0.4.5.336-5 (Sun Jun 4 14:25:45 2023)
==> Retrieving sources...
-> Branching lp:gloobus-preview...
You have not informed bzr of your Launchpad ID, and you must do this to
write to Launchpad or access private data. See "bzr help launchpad-login".
Branched 336 revisions.
==> Validating source files with sha256sums...

@kxxt thank you for noticing this change in behavior. Can you confirm that applying the original diff avoids requiring python-lazr-uri python-wadllib python-lazr-restfulclient and python-launchpadlib?
Comment by Levi Zim (kxxt) - Monday, 05 June 2023, 01:00 GMT
Yes, applying the original diff avoids these dependencies so it should be the preferred fix for this package.

But as long as "bzr+lp" is supported by makepkg, I think it is still a reason for moving python-launchpadlib from AUR to official repo.

Loading...