[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: Please update sight packaging

Hi Andreas,

I think, I hope, that this time this is the right one! :)

I added some more dependencies and remove some you added. I understand you added them to remove some CMake warnings, but normally we don't really care. iirc this is just because of some CMake files in VTK that are not as modular as we could expect. The executable files mentioned in the warnings are never used at build time nor at runtime.

I struggled a bit with pbuilder because we download some resource files during the configure step. I realized I needed to explicitly enable networking in ~/.pbuilderrc... I guess it is enabled on Debian build machines,  otherwise I would have noticed before, but I am just curious that you can confirm that to me.

I also had to add a new patch for some missing includes, we already fixed that upstream few days ago.

Please confirm that this builds fine on your side.

Cheers,


Dr. Flavien BRIDAULT
Director of Software Development
IRCAD France & IRCAD Africa

flavien.bridault@ircad.fr
Tél. : +33 (0)3 88 119 201
   IRCAD France
http://www.ircad.fr/
http://www.ircad.africa/

Suivez l'IRCAD sur  Facebook
 

IRCAD France
Hôpitaux Universitaires - 1, place de l'Hôpital - 67091 Strasbourg Cedex - FRANCE
Le 20/10/2021 à 09:33, Andreas Tille a écrit :
Hi Flavien,

Am Wed, Oct 20, 2021 at 08:45:34AM +0200 schrieb Flavien Bridault:

Thank you for this precious advice. The tip to run pdebuild when calling gbp
buildpackage definitely helps.

My next step was to try to build sight in pbuilder but I did not know how to
do so without camp being uploaded in unstable first...

I uploaded it yesterday:

$ apt-cache policy libcamp-dev
libcamp-dev:
  Installiert:           (keine)
  Installationskandidat: 0.8.4-1
  Versionstabelle:
     0.8.4-3 50
         50 http://deb.debian.org/debian unstable/main amd64 Packages
         50 http://incoming.debian.org/debian-buildd buildd-unstable/main amd64 Packages

A frequent trick is to add http://incoming.debian.org/debian-buildd to
your sources.list inside the pbuilder environment to get this sooner
before it hits unstable.


Actually I think I messed up when I rebased my changes above yours, because
of course, I have the list of dependencies since we have it on our internal
CI/CD pipeline. However I need to ensure that I provide a strict minimal
set. Let me fix this today.

Feel free to take the time you need. ;-)

Thanks a lot for maintaining sight

    Andreas.

Attachment: OpenPGP_signature
Description: OpenPGP digital signature

Reply to: