Re: [PATCH 00/12] dvbv5 scan tables for Brazil
Ah, you sent this to my +list e-mail, I have a backlog there of 10k mails due to a new job there now :) I pulled from your repository, and pushed it upstream. Thanks! Olliver On 08/31/2014 06:57 PM, Mauro Carvalho Chehab wrote: Ping. Em Sun, 27 Jul 2014 22:39:25 -0300 Mauro Carvalho Chehab m.che...@samsung.com escreveu: Hi Olliver,x Em Thu, 12 Jun 2014 21:59:13 +0200 Olliver Schinagl oliver+l...@schinagl.nl escreveu: Mauro, Sorry for my late reply!! I've tried to merge your patches, but besides some blank line errors, I got: fatal: cannot convert from true to UTF-8 I upgraded to git 2.0.0 and kept getting the error, atleast for 02. My git-foo is not strong enough to figure out whats going on here, or if the error is on my end or not. If it is on your end, could you re-send the patch series with the correct formatting? Weird... the patches are already in UTF-8. I just re-checked and applied them without any issue. I didn't even get any blank line errors. I suspect that something got mangled when you saved the patches from a file. I'm enclosing a tarball with all of them. As the patches are using UTF-8 encoding (due to the name of the Brazilian cities and due to the channel names that have accents), you need to be sure that you're using UTF-8 on your distribution. Here, I'm using: LANG=pt_BR.UTF-8 On my environment, to be sure that my charset is UTF-8. Anyway, you can also pull from my git tree directly with: git pull git://linuxtv.org/mchehab/dtv-scan-tables.git It will miss your SOB on the patches, though, but you can sign them either with git rebase or with git filter, with something like: $ git branch base $ git pull git://linuxtv.org/mchehab/dtv-scan-tables.git $ git filter-branch -f --msg-filter 'cat echo Signed-off-by: Olliver Schinagl oliver+l...@schinagl.nl' base..HEAD $ git branch -d base Regards, Mauro -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 00/12] dvbv5 scan tables for Brazil
Em Tue, 02 Sep 2014 20:59:13 +0200 Olliver Schinagl oliver+l...@schinagl.nl escreveu: Ah, you sent this to my +list e-mail, I have a backlog there of 10k mails due to a new job there now :) Ah! Well, a 10K backlog is not huge... mine is bigger, but what I do is to split emails on several inboxes, using a sieve filter. I pulled from your repository, and pushed it upstream. Thanks! Thanks! Olliver On 08/31/2014 06:57 PM, Mauro Carvalho Chehab wrote: Ping. Em Sun, 27 Jul 2014 22:39:25 -0300 Mauro Carvalho Chehab m.che...@samsung.com escreveu: Hi Olliver,x Em Thu, 12 Jun 2014 21:59:13 +0200 Olliver Schinagl oliver+l...@schinagl.nl escreveu: Mauro, Sorry for my late reply!! I've tried to merge your patches, but besides some blank line errors, I got: fatal: cannot convert from true to UTF-8 I upgraded to git 2.0.0 and kept getting the error, atleast for 02. My git-foo is not strong enough to figure out whats going on here, or if the error is on my end or not. If it is on your end, could you re-send the patch series with the correct formatting? Weird... the patches are already in UTF-8. I just re-checked and applied them without any issue. I didn't even get any blank line errors. I suspect that something got mangled when you saved the patches from a file. I'm enclosing a tarball with all of them. As the patches are using UTF-8 encoding (due to the name of the Brazilian cities and due to the channel names that have accents), you need to be sure that you're using UTF-8 on your distribution. Here, I'm using: LANG=pt_BR.UTF-8 On my environment, to be sure that my charset is UTF-8. Anyway, you can also pull from my git tree directly with: git pull git://linuxtv.org/mchehab/dtv-scan-tables.git It will miss your SOB on the patches, though, but you can sign them either with git rebase or with git filter, with something like: $ git branch base $ git pull git://linuxtv.org/mchehab/dtv-scan-tables.git $ git filter-branch -f --msg-filter 'cat echo Signed-off-by: Olliver Schinagl oliver+l...@schinagl.nl' base..HEAD $ git branch -d base Regards, Mauro -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 00/12] dvbv5 scan tables for Brazil
Hi Mauro, On Tue, 27 May 2014 13:50:20 -0300, Mauro Carvalho Chehab wrote: This patch series add the DTV scan tables for Brazilian ISDB-T and for the Brazilian Countys that have already digital TV. Thanks for the DVBv5 scan files. I had the attached draft patch sitting in my tree. I don't think it is ready to be committed yet, but probably worth sending now to discuss. How do we want to manage the migration from DVBv3 to DVBv5: 1) point in time migration from DVBv3 to DVBv5? 2) maintain both until DVBv5 is in widespread use? On a side note, I found a bug in dvb-format-convert; it cannot parse DVB-T2 DVBv3 scan files. Jon From fdcabb0802a4a40e257d54dbd5e5eba59b9820f7 Mon Sep 17 00:00:00 2001 From: Jonathan McCrohan jmccro...@gmail.com Date: Fri, 30 May 2014 01:14:42 +0100 Subject: [PATCH] Add Makefile to convert DVBv3 files to DVBv5 Signed-off-by: Jonathan McCrohan jmccro...@gmail.com --- Makefile | 25 + 1 file changed, 25 insertions(+) create mode 100644 Makefile diff --git a/Makefile b/Makefile new file mode 100644 index 000..fac40e7 --- /dev/null +++ b/Makefile @@ -0,0 +1,25 @@ +# Makefile for dtv-scan-tables (26 May 2014) +# Copyright 2014 Jonathan McCrohan jmccro...@gmail.com + +# The vast majority of the DVB scan files contained in this repository +# are DVBv3 scan files. This format has been deprecated in favor of the +# DVBv5 scan format. +# +# Use this makefile to convert the existing DVBv3 scan files to DVBv5 +# scan files until such time as DVBv5 scan format is in widespread use. +# +# Requires dvb-format-convert from v4l-utils. + +MKDIR = mkdir -p +DVBFORMATCONVERT = dvb-format-convert + +DVBFORMATCONVERT_CHANNEL_DVBV5 = -ICHANNEL -ODVBV5 + +DVBV3DIRS = atsc dvb-c dvb-s dvb-t +DVBV3CHANNELFILES = $(foreach dir,$(DVBV3DIRS),$(wildcard $(dir)/*)) + +DVBV5OUTPUTDIR = dvbv5 + +makedvbv5: + @$(foreach var,$(DVBV3DIRS), $(MKDIR) $(DVBV5OUTPUTDIR)/$(var);) + @$(foreach var,$(DVBV3CHANNELFILES), $(DVBFORMATCONVERT) $(DVBFORMATCONVERT_CHANNEL_DVBV5) $(var) $(DVBV5OUTPUTDIR)/$(var);) -- 2.0.0.rc2 pgpFgPUWQ3RBO.pgp Description: PGP signature
Re: [PATCH 00/12] dvbv5 scan tables for Brazil
Hi Jonathan, Em Fri, 30 May 2014 01:23:45 +0100 Jonathan McCrohan jmccro...@gmail.com escreveu: Hi Mauro, On Tue, 27 May 2014 13:50:20 -0300, Mauro Carvalho Chehab wrote: This patch series add the DTV scan tables for Brazilian ISDB-T and for the Brazilian Countys that have already digital TV. Thanks for the DVBv5 scan files. I had the attached draft patch sitting in my tree. I don't think it is ready to be committed yet, but probably worth sending now to discuss. Sure. How do we want to manage the migration from DVBv3 to DVBv5: 1) point in time migration from DVBv3 to DVBv5? 2) maintain both until DVBv5 is in widespread use? IMHO, the best strategy would be to convert all files at the tree to DVBv5, and add a Makefile target to produce the DVBv3 files and another one to install the DVBv5 files on a shared repository. That gives to distro maintainers the flexibility to have either one or two packages for each format, and let them to remove the dvbv3 when all apps on an specific distro would be using just the new format. On a side note, I found a bug in dvb-format-convert; it cannot parse DVB-T2 DVBv3 scan files. Well, at the time it was written, there were no DVB-T2 files. Feel free to send patches improving the library to also handle the DVB-T2 formats. Regards, Mauro signature.asc Description: PGP signature
