From: Hans de Goede <hdegoede@redhat.com>
To: Mauro Carvalho Chehab <mchehab@infradead.org>
Cc: Hans Verkuil <hverkuil@xs4all.nl>,
Linux Media Mailing List <linux-media@vger.kernel.org>,
Douglas Landgraf <dougsland@gmail.com>,
Brandon Philips <brandon@ifup.org>
Subject: Re: [ANNOUNCE] git tree repositories & libv4l
Date: Wed, 20 Jan 2010 19:50:16 +0100 [thread overview]
Message-ID: <4B575068.8050105@redhat.com> (raw)
In-Reply-To: <4B57241E.2060107@infradead.org>
Hi,
On 01/20/2010 04:41 PM, Mauro Carvalho Chehab wrote:
> Hans Verkuil wrote:
>>>> 4) v4l2-apps - I agree that splitting it could be a good idea, provided
>>>> that we find
>>>> a way to handle the few cases where we have "example" applications at
>>>> the media docs.
>>> Note that v4l2-apps also contains libv4l, it so happens that I've been
>>> discussing moving
>>> libv4l to its own git tree with Brandon Philips. Preferably to a place
>>> which also offers
>>> some form of bug tracking. The advantages of having libv4l in its own tree
>>> are:
>>>
>>> -it is maintained independent of the hg tree anyways
>>> -it has regular versioned tarbal releases, it would be good to be able to
>>> tag these
>>> inside the used scm, which is hard to do when the scm is shared with
>>> other unrelated
>>> code which does not end up in said tarballs
>>> -this means having a much smaller tree making it easier to clone
>>> -no longer having an often old (stale) libv4l in the master hg repository
>>> (this is partially my fault as I should send pull requests for libv4l
>>> moe often,
>>> but why all this synchronization overhead when its independent anyways)
>>>
>>> As said when discussing this with Brandon we were thinking about using
>>> something
>>> like github, as that offers bug tracking too. But I can understand if you
>>> would prefer
>>> to keep libv4l at linuxtv.org .
>
> Hans G.,
>
> I prefer to keep it at linuxtv.org, and together with v4l2-apps. The rationale
> is that there are applications that are dependent on libv4l (like v4l2grab).
> We're also about to commit a gtk webcam application based on libv4l. So, IMO,
> the better is to keep all of those applications together.
>
> As we're discussing about having a separate tree for v4l2-apps, maybe the better
> is to port it to -git (in a way that we can preserve the log history).
>
Having a separate tree for v4l2-apps would work for me. If possible with direct
commit / push rights, given that I'm doing 90% of the libv4l work.
>> (except for bug tracking).
>
> Well, I don't see why not adding a bug tracking system for v4l2-apps an dvb-apps,
> if needed. Are you thinking on something specific?
>
If I can put in wishes I would like bugzilla :)
>> We would need to do
>> some rearranging in the directory structure of v4l2-apps, though.
>
> Yes. Maybe we can move the tools that aren't meant to be used on distros on a separate
> dir, like contrib, having a separate make install for building them.
>
> Also, we need to use some config tool like autoconf that will seek for dependencies
> and or require the needed ones or not compile the applications that depends on some
> library.
>
Ugh, I'm no fan of autoconf, but I can see this being handy, any volunteers for
doing this bit ?
>> It is a bit of a mix of production code like libv4l and v4l2-ctl, v4l2-dbg.
>> cx18-ctl and ivtv-ctl, and more test or debug oriented tools. The latter
>> do not need to be packaged by distros.
>
> Yes.
>
>> Personally I would prefer to keep v4l2-apps on linuxtv.org. But the strict
>> commit procedure that we have for the main v4l-dvb repo can be relaxed
>> here.
>
> What do you mean by a "relaxed" one? Currently, there's no defined procedure for
> the applications there: several different CodingStyles were used on the different
> applications and no standard criteria were used to ack/nack changes there.
>
> IMO, we should go on the opposite direction: we need to have some standard rules
> also for it (but, it can be more relaxed than what we have in kernel).
>
Ack, we need to at least have rules, like should build on a modern distro
without issues (if the deps are there), etc.
> For sure, one rule we need to define is what criteria will be used to classify
> an application as something that will be compiled/installed by default, and what
> applications are development-oriented applications. On some cases, this is clear
> (for example, the API compliance test applications are developer-oriented, while
> libv4l is a standard user-oriented one). However, a debug application (like v4l2-dbg)
> is a development application, but it may be nice to have it available at the
> distros, to help users to help check/report problems).
>
Ack, I too think having v4l2-dbg available to end users could come in very handy to
remote debug stuff.
> It may also be useful to define a minimum set of coding style, like how applications
> should be indented
>
>> So it should be possible for you to have commit rights so you can
>> use it as your master repository.
>
> Maybe the better owner for v4l2-apps would be Hans G., since most of the changes there
> are related to libv4l.
>
I wouldn't mind giving this a try.
> On the experiences we had with v4l-dvb tree, it is not a good idea to allow multiple
> people to commit at the master repository, since, when a conflict rises between two
> different developers, this can cause lots of heat. Also, it is easy to corrupt a tree,
> as a push with -f flag can remove (or hide, on -git) the objects inserted by someone else.
>
I've different experience in the projects with git I've used, as long as there are some
governance rules (like never ever push -f, always do a rebase fix your stuff and then
push, and if something else got in in the window in between rebase again, etc.).
Regards,
Hans
next prev parent reply other threads:[~2010-01-20 20:28 UTC|newest]
Thread overview: 91+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-01-19 5:34 [ANNOUNCE] git tree repositories Mauro Carvalho Chehab
2010-01-19 7:53 ` Hans Verkuil
2010-01-19 8:10 ` Patrick Boettcher
2010-01-19 11:20 ` Johannes Stezenbach
2010-01-19 11:49 ` Patrick Boettcher
2010-01-19 12:44 ` Mauro Carvalho Chehab
2010-01-20 8:04 ` Markus Heidelberg
2010-01-20 15:11 ` Mauro Carvalho Chehab
2010-01-19 11:55 ` Mauro Carvalho Chehab
2010-01-19 23:38 ` Andy Walls
2010-01-20 1:10 ` hermann pitton
2010-01-20 3:29 ` Andy Walls
2010-01-20 3:29 ` Mauro Carvalho Chehab
2010-01-20 4:08 ` Andy Walls
2010-01-20 15:05 ` Mauro Carvalho Chehab
2010-01-20 11:43 ` BOUWSMA Barry
2010-01-20 10:19 ` BOUWSMA Barry
2010-01-19 11:08 ` Mauro Carvalho Chehab
2010-01-20 8:36 ` [ANNOUNCE] git tree repositories & libv4l Hans de Goede
2010-01-20 8:55 ` Hans Verkuil
2010-01-20 15:41 ` Mauro Carvalho Chehab
2010-01-20 18:50 ` Hans de Goede [this message]
2010-01-20 21:07 ` Brandon Philips
2010-01-21 2:07 ` Mauro Carvalho Chehab
2010-01-21 2:46 ` Brandon Philips
2010-01-21 7:34 ` Hans Verkuil
2010-01-21 20:15 ` Mauro Carvalho Chehab
2010-01-23 17:24 ` Hans de Goede
2010-02-22 22:54 ` Brandon Philips
2010-02-22 23:26 ` Hans Verkuil
2010-02-22 23:38 ` Brandon Philips
2010-02-23 0:41 ` Mauro Carvalho Chehab
2010-02-23 1:12 ` Mauro Carvalho Chehab
2010-02-23 8:04 ` Brandon Philips
2010-02-23 13:37 ` Mauro Carvalho Chehab
2010-02-23 9:45 ` Manu Abraham
2010-02-23 11:20 ` Mauro Carvalho Chehab
2010-02-24 2:32 ` hermann pitton
2010-02-23 11:20 ` Manu Abraham
2010-02-23 8:49 ` Hans de Goede
2010-02-23 9:01 ` Hans Verkuil
2010-02-23 9:23 ` Hans de Goede
2010-02-23 9:38 ` Manu Abraham
2010-02-23 12:21 ` Mauro Carvalho Chehab
2010-02-23 13:07 ` Manu Abraham
2010-02-23 13:36 ` Mauro Carvalho Chehab
2010-02-23 14:09 ` Manu Abraham
2010-02-23 12:10 ` Mauro Carvalho Chehab
2010-02-23 11:49 ` Mauro Carvalho Chehab
2010-02-23 15:37 ` Mauro Carvalho Chehab
2010-02-23 15:51 ` Hans de Goede
2010-02-24 0:58 ` Mauro Carvalho Chehab
2010-02-24 12:55 ` Hans de Goede
2010-02-24 13:40 ` Hans Verkuil
2010-02-24 13:42 ` Mauro Carvalho Chehab
2010-02-24 14:32 ` Brandon Philips
2010-02-25 10:52 ` Hans de Goede
2010-02-24 6:05 ` Brandon Philips
2010-02-24 12:44 ` Hans de Goede
2010-02-24 13:26 ` Mauro Carvalho Chehab
2010-02-24 14:29 ` Patrick Boettcher
2010-02-25 4:55 ` Douglas Schilling Landgraf
2010-01-21 7:23 ` Hans Verkuil
2010-01-21 20:04 ` Mauro Carvalho Chehab
2010-01-23 17:28 ` Hans de Goede
2010-01-24 0:42 ` Mauro Carvalho Chehab
2010-01-25 16:03 ` Hans de Goede
2010-01-20 9:43 ` Laurent Pinchart
2010-01-20 9:54 ` Paulo Assis
2010-01-20 10:43 ` libwecam & uvcdynctrl (was Re: [ANNOUNCE] git tree repositories & libv4l) Hans de Goede
2010-01-19 15:54 ` [ANNOUNCE] git tree repositories Douglas Schilling Landgraf
2010-01-19 8:04 ` Laurent Pinchart
2010-01-19 11:12 ` Mauro Carvalho Chehab
2010-01-19 11:50 ` Laurent Pinchart
2010-01-19 12:36 ` Mauro Carvalho Chehab
2010-01-19 10:04 ` Devin Heitmueller
2010-01-19 11:52 ` Patrick Boettcher
2010-01-19 12:39 ` Mauro Carvalho Chehab
2010-01-19 12:16 ` Mauro Carvalho Chehab
2010-01-19 21:21 ` Bob Cunningham
2010-01-19 22:37 ` hermann pitton
2010-01-20 13:54 ` Laurent Pinchart
2010-01-20 15:00 ` Mauro Carvalho Chehab
2010-01-19 12:56 ` Laurent Pinchart
2010-01-19 13:07 ` Mauro Carvalho Chehab
2010-01-19 13:12 ` Laurent Pinchart
2010-01-19 21:59 ` Johannes Stezenbach
2010-01-21 2:19 ` Mauro Carvalho Chehab
2010-01-21 2:53 ` Trent Piepho
2010-01-21 3:09 ` Mauro Carvalho Chehab
2010-01-20 19:09 ` Guennadi Liakhovetski
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4B575068.8050105@redhat.com \
--to=hdegoede@redhat.com \
--cc=brandon@ifup.org \
--cc=dougsland@gmail.com \
--cc=hverkuil@xs4all.nl \
--cc=linux-media@vger.kernel.org \
--cc=mchehab@infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox