FAQ Search Today's Posts Mark Forums Read
» Video Reviews

» Linux Archive

Linux-archive is a website aiming to archive linux email lists and to make them easily accessible for linux users/developers.


» Sponsor

» Partners

» Sponsor

Go Back   Linux Archive > Debian > Debian dpkg

 
 
LinkBack Thread Tools
 
Old 01-31-2012, 09:09 PM
"Adam D. Barratt"
 
Default Multi-arch branches

On Tue, 2012-01-31 at 15:00 -0600, Jonathan Nieder wrote:
> Guillem Jover wrote:
>
> > I have a problem with merging unreviewed code, though, that among other
> > things might and do have design issues, as witnessed in the past. I also
> > have not seen Cyril jump to do any kind of code review during all this
> > time as quickly as he jumped to pester and with threats to NMU a new
> > *upstream release*, w/o having been involved with it at *all*, with
> > known issues, regardless of maintainer disapproval...
>
> To be honest, I think it's okay. KiBi is on the release team and

On, but not acting on behalf of, fwiw.

Regards,

Adam


--
To UNSUBSCRIBE, email to debian-dpkg-REQUEST@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmaster@lists.debian.org
Archive: 1328047759.7556.6.camel@jacala.jungle.funky-badger.org">http://lists.debian.org/1328047759.7556.6.camel@jacala.jungle.funky-badger.org
 
Old 01-31-2012, 10:49 PM
Raphael Hertzog
 
Default Multi-arch branches

Hi,

On Tue, 31 Jan 2012, Guillem Jover wrote:
> I have a problem with merging unreviewed code, though, that among other
> things might and do have design issues, as witnessed in the past. I also
> have not seen Cyril jump to do any kind of code review during all this
> time as quickly as he jumped to pester and with threats to NMU a new
> *upstream release*, w/o having been involved with it at *all*, with
> known issues, regardless of maintainer disapproval...
>
> One of the many reasons this past year has been pretty distressful, to
> be honest.

To be honest, reviewing 66 commits and finding design issues in them
doesn't require 10 months.

You might not care about the schedule of getting multiarch working into
Debian but some people do (including me, many ARM porters, and several
persons on the release team given the past discussions). I think that
you're aware that you have put yourself on the critical path and that
you're the one causing the delay in the completion of this project.

If I were you, I would try to not stay on the critical path: I would do
a public code review pointing out the design issues that you identified
and I would ask the submitter of the code to fix the issues. After that
you can freely help the submitter and do yourself some of the suggested
fixes (and code cleanup/refactoring) but at least other people can step in
to get things moving forward.

Right now, I have been unable to do anything to help you since I submitted
the code except reviewing and testing your changes... changes that I
detect manually by trying to fetch your private repository from time to
time and by running a git diff between the previous copy.

It's fine to work alone when it's your own work, but when you work on code
that has been submitted by others, it would be nice to cooperate more
closely with them. (Even more when it's the co-maintainer and that he's
willing to learn from his mistakes so that at some point in the future you
can trust him... because right now you're not trusting anyone except you.
So it's a bit strange to suggest that a code review by Cyril would have
helped.)

> > I used 1.16.2~ on purpose because we have users testing the multiarch branch
> > out of the dpkg-test APT repository which has this kind of versioning.
>
> And that's why that belongs in such branch, for master it does not
> make sense because there's no such tag there anyway, I've been meaning
> to remove the tildes before uploading the final 1.16.2 in any case.

Ok.

Cheers,
--
Raphaël Hertzog ◈ Debian Developer

Pre-order a copy of the Debian Administrator's Handbook and help
liberate it: http://debian-handbook.info/liberation/


--
To UNSUBSCRIBE, email to debian-dpkg-REQUEST@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmaster@lists.debian.org
Archive: 20120131234915.GE3962@rivendell.home.ouaza.com">ht tp://lists.debian.org/20120131234915.GE3962@rivendell.home.ouaza.com
 
Old 02-01-2012, 04:52 AM
Guillem Jover
 
Default Multi-arch branches

On Mon, 2012-01-23 at 09:04:12 +0100, Raphael Hertzog wrote:
> I just had to update the test suite to cope with the rename of
> ${virt:Package-Spec} to ${binary:Package}.

BTW, forgot to mention, I had fixed this and the
--print-foreign-architectures changes in my pkg-tests.git repo.

regards,
guillem


--
To UNSUBSCRIBE, email to debian-dpkg-REQUEST@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmaster@lists.debian.org
Archive: 20120201055220.GA8316@gaara.hadrons.org">http://lists.debian.org/20120201055220.GA8316@gaara.hadrons.org
 

Thread Tools




All times are GMT. The time now is 07:38 AM.

VBulletin, Copyright ©2000 - 2014, Jelsoft Enterprises Ltd.
Content Relevant URLs by vBSEO ©2007, Crawlability, Inc.
Copyright 2007 - 2008, www.linux-archive.org