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

Re: Packaging for cbind/cview



On Tue, Apr 06, 2021 at 09:20:55AM -0700, Micheal Waltz wrote:
> On Mon, Apr 05, 2021 at 01:49:26PM +0530, Nilesh Patra wrote:
> > > > > I've re-packaged cbind with the new upstream and it's on mentors
> > > > > and a new RFS is created if you could please review.
> > > > 
> > > > Can you please push this to salsa?
> > > 
> > > The salsa repo is up-to-date. I just added the debian/gitlab-ci.yml
> > > a few minutes ago.
> > > 
> > > https://salsa.debian.org/go-team/packages/golang-code.rocketnine-tslocum-cbind
> > 
> > Alright, I'll take a look in a while.
> 
> Thank you for the upload to NEW.
> 
> My next step here is packaging cview, which uses cbind as a dependency.
> 
> Can I build cview using --extra-package and pointing to the local cbind
> package. Then do an upload to mentors/salsa and follow the usual upload
> workflow?

Yes

> The same thing with Amfora then, when cview is packaged I can then do
> --extra-package for the deps that are in NEW and not fully uploaded into
> the archive?

Yes, that is exactly how I'll build and sponsor the upload for you as
well.

> In regards to Amfora, the current HEAD in github is updated to the new
> rocketnine repo for cbind/cview. However the 1.8.0 release is from Feb
> that does not have this change.
> I spoke with upstream about a timeline for a 1.9.0 release and there is
> no plans for it soon. So in order to package Amfora with these newer
> godeps we'll need to package from HEAD instead of a release.

Why is that needed?
Simply patch it to use new path? I think in migration, the only change
would've be in import pth, and nothing else.

Since you mention HEAD, simply cherry pick the relevant commit for
codenine rocket change, and apply as a debian patch[1]

> Is this okay? I'd prefer to use a point release when possible but would
> like to use a newer codebase for the package instead of one from a few
> months ago.

I'd always suggest prefering release versions, see the reply above.

[1]: https://wiki.debian.org/UsingQuilt

Nilesh

Attachment: signature.asc
Description: PGP signature


Reply to: