public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
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

  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