Re: [PATCH 01/10] alsa_stream: port changes made on xawtv3

[Mauro Carvalho Chehab <mchehab@redhat.com>]

Em 06-09-2011 18:18, Devin Heitmueller escreveu:
> On Tue, Sep 6, 2011 at 3:12 PM, Mauro Carvalho Chehab
> <mchehab@redhat.com> wrote:
>>> From a practical standpoint, the Ubuntu folks have the original tvtime
>>> tarball and all their changes in one patch, which is clearly a bunch
>>> of patches that are mashed together probably in their build system. I
>>> need to reach out to them to find where they have an actual SCM tree
>>> or the individual patches.  They've got a bunch of patches which would
>>> be good to get into a single tree (autobuild fixes, cross-compilation,
>>> locale updates, etc).
>>
>> Yeah, it seems interesting. Maybe we can get something from this place:
>>        http://packages.qa.debian.org/t/tvtime.html
>>
>> The maintainer there seems to be:
>>        http://qa.debian.org/developer.php?login=bartm@debian.org
> 
> I reached out to the Ubuntu maintainer; we'll see if he gets back to
> me.  From what I can tell it seems like Debian is actually taking the
> patches from Ubuntu (yes, I realize this is backwards from their
> typical process where Ubuntu bases their stuff on Debian).

Good!

>>> In the long term I have no real issue with the LinuxTV group being the
>>> official maintainer of record.  I've got lots of ideas and things I
>>> would like to do to improve tvtime, but in practice I've done a pretty
>>> crappy job of maintaining the source (merging patches, etc) at this
>>> point.
>>
>> Putting it on a common place and giving permissions to a group of people
>> is interesting, as none of us are focused on userspace, so we all
>> have a very limited amount of time for dealing with userspace applications.
>>
>> By giving commit rights to a group of developers, it ends that more
>> developers will contribute, speeding up the development.
>>
>> That was what happened with v4l-utils and, on a minor scale, with xawtv3.
>>
>> If you're ok with that, I can set a tvtime git repository at LinuxTV,
>> cloning the tree I've created there already (it is a pure conversion
>> of your tree from mercurial into git, if I remove the patches I've
>> done so far from your clone), giving you the ownership of the new tree,
>> and marking it as a shared repository.
> 
> I have no problem with this.  Let's set it up.

Ok. The repository is here:
	http://git.linuxtv.org/tvtime.git

In thesis, everything is set for group usage. Please let me know if
you experience any troubles with it.

>> I have already all set there to allow shared access to the repository
>> (in opposite to -hg, git works really cool with shared repositories).
> 
> I actually haven't hosted any git repos on linuxtv.org before.  I'm
> assuming my ssh public key got copied over from when I was hosting hg
> repos there?

The same key is used, whatever you're committing to cvs, hg or git. The
maintenance application for git is called git-menu.

>> We can later add permissions to the developers interested on helping
>> the tvtime maintenance that you agree to add.
> 
> Sounds good.

>From my side, I'm interested on helping with it.

When I have some time, I'd like to fix a few issues with it. 

For example, there's a local cable operator that broadcasts some channels 
with PAL/M and others with NTSC/M (not a big deal for STBs and TV sets, as
almost all support both standards here).

However, tvtime, needs to be restarted every time it changes from one to 
the other, and it is not possible to set a per-channel standard. To be 
worse, when tvtime is restarted, it doesn't honor the "-d" option, with
means that it will open my laptop's webcam instead of the TV card.
> 
> As said earlier, Kernel Labs never really wanted to be the maintainer
> for tvtime - we did it because nobody else wanted to (and vektor never
> responded to emails I sent him offering to help).  That said, a
> community oriented approach is probably the best for everybody
> involved.
> 
> I'll probably be looking in the next couple of weeks to write some
> fresh content for a tvtime website.  The stuff on
> tvtime.sourceforge.net is so dated almost none of it still applies.

Yeah, that makes sense.
> Thanks,
> 
> Devin
>