[Bug 1358293] Review Request: bcg729 - Opensource implementation of the G.729 codec
https://bugzilla.redhat.com/show_bug.cgi?id=1358293 --- Comment #29 from Peter Lemenkov --- (In reply to Tom "spot" Callaway from comment #26) > Removing the FE-Legal block on this package. G.729 implementations are > permissible in Fedora now. Awesome news! -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1358293] Review Request: bcg729 - Opensource implementation of the G.729 codec
https://bugzilla.redhat.com/show_bug.cgi?id=1358293 Sandro Mani changed: What|Removed |Added Status|NEW |CLOSED Resolution|--- |DUPLICATE Last Closed|2016-07-27 03:57:14 |2017-12-20 10:15:55 --- Comment #28 from Sandro Mani --- Mhh looks like I can't re-use this review (older than 60 days), need a re-review -> 1527953 *** This bug has been marked as a duplicate of bug 1527953 *** -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1358293] Review Request: bcg729 - Opensource implementation of the G.729 codec
https://bugzilla.redhat.com/show_bug.cgi?id=1358293 --- Comment #27 from Sandro Mani --- Thanks! I'll go ahead and import the package then. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1358293] Review Request: bcg729 - Opensource implementation of the G.729 codec
https://bugzilla.redhat.com/show_bug.cgi?id=1358293 Tom "spot" Callaway changed: What|Removed |Added Blocks|182235 (FE-Legal) | --- Comment #26 from Tom "spot" Callaway --- Removing the FE-Legal block on this package. G.729 implementations are permissible in Fedora now. Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=182235 [Bug 182235] Fedora Legal Tracker -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1358293] Review Request: bcg729 - Opensource implementation of the G.729 codec
https://bugzilla.redhat.com/show_bug.cgi?id=1358293 nucleo changed: What|Removed |Added CC||alekc...@googlemail.com -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1358293] Review Request: bcg729 - Opensource implementation of the G.729 codec
https://bugzilla.redhat.com/show_bug.cgi?id=1358293 Peter Lemenkov changed: What|Removed |Added Whiteboard||NotReady -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1358293] Review Request: bcg729 - Opensource implementation of the G.729 codec
https://bugzilla.redhat.com/show_bug.cgi?id=1358293 Peter Lemenkov changed: What|Removed |Added Status|CLOSED |NEW Resolution|NOTABUG |--- Keywords||Reopened --- Comment #25 from Peter Lemenkov --- I'd like to reopen this considering recent legal changes mentioned above. Just to make sure it will be on the radar. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1358293] Review Request: bcg729 - Opensource implementation of the G.729 codec
https://bugzilla.redhat.com/show_bug.cgi?id=1358293 Peter Lemenkov changed: What|Removed |Added CC||pleme...@redhat.com --- Comment #24 from Peter Lemenkov --- (In reply to Tom "spot" Callaway from comment #22) > Does bcg729 implement Annex C+, D, E, F, G, H, or I? Tom, please take a look at the Patent's owner announcement: http://www.sipro.com/G729.html G.729 – IMPORTANT INFORMATION As of January 1, 2017 the patent terms of most Licensed Patents under the G.729 Consortium have expired. With regard to the unexpired Licensed Copyrights and Licensed Patents of the G.729 Consortium Patent License Agreement, the Licensors of the G.729 Consortium, namely Orange SA, Nippon Telegraph and Telephone Corporation and Université de Sherbrooke (“Licensors”) have agreed to license the same under the existing terms on a royalty-free basis starting January 1, 2017. For current Licensees of the G.729 Consortium Patent License Agreement, no reports and no payments will be due for Licensed Products Sold or otherwise distributed as of January 1, 2017. For other companies selling G.729 compliant products and that are not current Licensees of the G.729 Consortium, there is no need to execute a G.729 Consortium Patent License Agreement since Licensors have agreed to license the unexpired Licensed Copyrights and Licensed Patents of the G.729 Consortium Patent License Agreement under the existing terms on a royalty-free basis starting January 1, 2017. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1358293] Review Request: bcg729 - Opensource implementation of the G.729 codec
https://bugzilla.redhat.com/show_bug.cgi?id=1358293 --- Comment #23 from Peter Lemenkov --- (In reply to Tom "spot" Callaway from comment #22) > Does bcg729 implement Annex C+, D, E, F, G, H, or I? No. It implements only a basic A profile (integer codec) also known as G.729a, and B profile, also known as G.729b. B profile is essentially the same as A profile, but with CN (comfort noise), VAD (Voice Auto Detection), and DTX (Discontinuous Transmission) packets handling implemented. "A" profile decoder can handle "B" compliant stream by skipping unknown packets related to CN/VAD/DTX. "B" decoder can handle "A" compliant data w/o any issues. A generic CN/VAD/DTX payload for codecs w/o support (for example - G.711 / G.722) is described in RFC 3389: https://tools.ietf.org/html/rfc3389 Please also read G.729 wikipedia entry carefully. Especially non-technical part below the compatibility matrix and annex descriptions. Just for the record - what people usually call G.729 is a different codec, although compatible, anyone can hardly find in the wild. Other G.729 extensions (C, D, etc) are also not so easy to find. If a codec can encode and decode G.729a stream this means that both parties can hear each other (unknown packets can be skipped, and any bigger "annex" codec can provide compatible output - fixed point, 8 kbit/s, no extensions). G.729.1 is a totally different incompatible codec, and outside of the scope of this library. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1358293] Review Request: bcg729 - Opensource implementation of the G.729 codec
https://bugzilla.redhat.com/show_bug.cgi?id=1358293 --- Comment #22 from Tom "spot" Callaway --- Does bcg729 implement Annex C+, D, E, F, G, H, or I? -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1358293] Review Request: bcg729 - Opensource implementation of the G.729 codec
https://bugzilla.redhat.com/show_bug.cgi?id=1358293 Peter Lemenkov changed: What|Removed |Added CC||lemen...@gmail.com --- Comment #21 from Peter Lemenkov --- (In reply to Itamar Reis Peixoto from comment #17) > I mailed spot [1], if he invite us to remove this package from fedora then > We send it to rpmfusion. > > [1] - https://fedoraproject.org/wiki/User:Spot (In reply to Tom "spot" Callaway from comment #18) > Uh, yeah. This cannot go into Fedora. Sorry. I'd like to reopen discussion about the validity of 3rd party claims about patent ownership covering some VoIP codec algorithms related to this codec. Upstream of this library believes that any patents covering most widely used A/B annexes of the codec have just expired (nov 2016). https://github.com/BelledonneCommunications/bcg729/commit/2f6b033 Tom, could you please ask Fedora Legal to add this on their radar? Actually all G.72x codecs have a very lengthy history dating back to late 80ies - early 90ies, so chances are high that some of them are no longer covered by unexpired patents. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1358293] Review Request: bcg729 - Opensource implementation of the G.729 codec
https://bugzilla.redhat.com/show_bug.cgi?id=1358293 Sandro Mani changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution|--- |NOTABUG Flags|needinfo?(manisandro@gmail. | |com)| Last Closed||2016-07-27 03:57:14 --- Comment #20 from Sandro Mani --- Submitted for review as RFBZ #4155. Closing this bug. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org
[Bug 1358293] Review Request: bcg729 - Opensource implementation of the G.729 codec
https://bugzilla.redhat.com/show_bug.cgi?id=1358293 Itamar Reis Peixoto changed: What|Removed |Added CC||manisan...@gmail.com Flags||needinfo?(manisandro@gmail. ||com) --- Comment #19 from Itamar Reis Peixoto --- would you like to send it for inclusion in rpmfusion ? -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org
[Bug 1358293] Review Request: bcg729 - Opensource implementation of the G.729 codec
https://bugzilla.redhat.com/show_bug.cgi?id=1358293 Tom "spot" Callaway changed: What|Removed |Added Status|ON_QA |ASSIGNED CC||tcall...@redhat.com --- Comment #18 from Tom "spot" Callaway --- Uh, yeah. This cannot go into Fedora. Sorry. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org
[Bug 1358293] Review Request: bcg729 - Opensource implementation of the G.729 codec
https://bugzilla.redhat.com/show_bug.cgi?id=1358293 --- Comment #17 from Itamar Reis Peixoto --- I mailed spot [1], if he invite us to remove this package from fedora then We send it to rpmfusion. [1] - https://fedoraproject.org/wiki/User:Spot -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org
[Bug 1358293] Review Request: bcg729 - Opensource implementation of the G.729 codec
https://bugzilla.redhat.com/show_bug.cgi?id=1358293 --- Comment #16 from Sandro Mani --- I fear I was completely unaware of the patent situation surrounding this codec, and missed the corresponding line in the README file: "Licensing: The source code is licensed under GPLv2. ITU G729 usage is governed by a patent license to be acquired from Sipro Lab" I can't find any hints that the patent situation has changed, in which case this package needs to be removed from the distribution. Apologies for missing this, multimedia stuff always requires special attention... -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org
[Bug 1358293] Review Request: bcg729 - Opensource implementation of the G.729 codec
https://bugzilla.redhat.com/show_bug.cgi?id=1358293 --- Comment #15 from Fedora Update System --- bcg729-1.0.0-2.fc24 has been pushed to the Fedora 24 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2016-8e02805a7a -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org
[Bug 1358293] Review Request: bcg729 - Opensource implementation of the G.729 codec
https://bugzilla.redhat.com/show_bug.cgi?id=1358293 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #14 from Fedora Update System --- bcg729-1.0.0-2.fc23 has been pushed to the Fedora 23 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2016-2c219c2a4d -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org
[Bug 1358293] Review Request: bcg729 - Opensource implementation of the G.729 codec
https://bugzilla.redhat.com/show_bug.cgi?id=1358293 Itamar Reis Peixoto changed: What|Removed |Added Blocks||182235 (FE-Legal) Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=182235 [Bug 182235] Fedora Legal Tracker -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org
[Bug 1358293] Review Request: bcg729 - Opensource implementation of the G.729 codec
https://bugzilla.redhat.com/show_bug.cgi?id=1358293 Greg Bailey changed: What|Removed |Added CC||gbai...@lxpro.com --- Comment #13 from Greg Bailey --- Has the patent for Annex A of ITU-T G.729 finally expired? Did anyone run this by legal to make sure there's no issues with what's described at: https://fedoraproject.org/wiki/Software_Patents -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org
[Bug 1358293] Review Request: bcg729 - Opensource implementation of the G.729 codec
https://bugzilla.redhat.com/show_bug.cgi?id=1358293 --- Comment #12 from Fedora Update System --- bcg729-1.0.0-2.fc24 has been submitted as an update to Fedora 24. https://bodhi.fedoraproject.org/updates/FEDORA-2016-8e02805a7a -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org
[Bug 1358293] Review Request: bcg729 - Opensource implementation of the G.729 codec
https://bugzilla.redhat.com/show_bug.cgi?id=1358293 Fedora Update System changed: What|Removed |Added Status|POST|MODIFIED -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org
[Bug 1358293] Review Request: bcg729 - Opensource implementation of the G.729 codec
https://bugzilla.redhat.com/show_bug.cgi?id=1358293 --- Comment #11 from Fedora Update System --- bcg729-1.0.0-2.fc23 has been submitted as an update to Fedora 23. https://bodhi.fedoraproject.org/updates/FEDORA-2016-2c219c2a4d -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org
[Bug 1358293] Review Request: bcg729 - Opensource implementation of the G.729 codec
https://bugzilla.redhat.com/show_bug.cgi?id=1358293 --- Comment #10 from Jon Ciesla --- Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/rpms/bcg729 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org
[Bug 1358293] Review Request: bcg729 - Opensource implementation of the G.729 codec
https://bugzilla.redhat.com/show_bug.cgi?id=1358293 --- Comment #9 from Sandro Mani --- (Followup remark: orpt is only used when building the mediastreamer plugin, hence removed again) -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org
[Bug 1358293] Review Request: bcg729 - Opensource implementation of the G.729 codec
https://bugzilla.redhat.com/show_bug.cgi?id=1358293
--- Comment #8 from Sandro Mani ---
> * add --disable-silent-rules to %configure
> * %{name} is not used everywhere in %files
Done
> * file bug to upstream that AC_PROG_LIBTOOL is obsoleted and they should use
> LT_INIT
> * file bug to upstream about incorrect FSF address
Sent to linphone-develop...@nongnu.org, should appear at [1] some time.
Thanks for the review!
[1] http://lists.nongnu.org/archive/html/linphone-developers/2016-07/index.html
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org
[Bug 1358293] Review Request: bcg729 - Opensource implementation of the G.729 codec
https://bugzilla.redhat.com/show_bug.cgi?id=1358293 Igor Gnatenko changed: What|Removed |Added Status|ASSIGNED|POST -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org
[Bug 1358293] Review Request: bcg729 - Opensource implementation of the G.729 codec
https://bugzilla.redhat.com/show_bug.cgi?id=1358293
Igor Gnatenko changed:
What|Removed |Added
Flags||fedora-review+
--- Comment #7 from Igor Gnatenko ---
> - Enabling mediastreamer results in the library built as a mediastreamer
> plugin, which is what is desired for twinkle
Unfortunately I don't understand this..
* add --disable-silent-rules to %configure
* %{name} is not used everywhere in %files
* file bug to upstream that AC_PROG_LIBTOOL is obsoleted and they should use
LT_INIT
* file bug to upstream about incorrect FSF address
Nothing else to complain about.
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org
[Bug 1358293] Review Request: bcg729 - Opensource implementation of the G.729 codec
https://bugzilla.redhat.com/show_bug.cgi?id=1358293 --- Comment #6 from Sandro Mani --- - Test not enabled since it downloads data from the internet, but this data yelds a checksum failure - Enabling mediastreamer results in the library built as a mediastreamer plugin, which is what is desired for twinkle Rest done: Spec URL: https://smani.fedorapeople.org/review/bcg729.spec SRPM URL: https://smani.fedorapeople.org/review/bcg729-1.0.0-2.fc25.src.rpm %changelog * Wed Jul 20 2016 Sandro Mani - 1.0.0-2 - Move autoreconf to build - BR: pkgconfig(ortp) - License is GPLv2+ - Change command to delete *.la files - Use %%name in %%files - Don't fix FSF addresses -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org
[Bug 1358293] Review Request: bcg729 - Opensource implementation of the G.729 codec
https://bugzilla.redhat.com/show_bug.cgi?id=1358293 --- Comment #5 from Igor Gnatenko --- (In reply to Sandro Mani from comment #4) > Concering FSF address: > https://fedoraproject.org/wiki/Common_Rpmlint_issues#incorrect-fsf-address > > So COPYING should not be patched, rest can but does not have to be. so don't waste CPU cycles on doing sed, just report bug in upstream and forget about it until next release ;) -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org
[Bug 1358293] Review Request: bcg729 - Opensource implementation of the G.729 codec
https://bugzilla.redhat.com/show_bug.cgi?id=1358293 --- Comment #4 from Sandro Mani --- Concering FSF address: https://fedoraproject.org/wiki/Common_Rpmlint_issues#incorrect-fsf-address So COPYING should not be patched, rest can but does not have to be. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org
[Bug 1358293] Review Request: bcg729 - Opensource implementation of the G.729 codec
https://bugzilla.redhat.com/show_bug.cgi?id=1358293
Itamar Reis Peixoto changed:
What|Removed |Added
CC||ita...@ispbrasil.com.br
--- Comment #3 from Itamar Reis Peixoto ---
why not ?
find %{buildroot} -name '*.la' -delete
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org
[Bug 1358293] Review Request: bcg729 - Opensource implementation of the G.729 codec
https://bugzilla.redhat.com/show_bug.cgi?id=1358293 --- Comment #2 from Sandro Mani --- >> # Fix FSF addresses > don't do that, just report bug in upstream and note link somewhere around > License tag. Any particular reason? In the past, I've been instructed to fix the FSF addr. From my understanding, it also makes sense - or does this violate "shipping a license file different from upstream" in you opinion? Now whether this needs to be done with seds or with a huge patch is another issue, but plenty of packages are doing this. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org
[Bug 1358293] Review Request: bcg729 - Opensource implementation of the G.729 codec
https://bugzilla.redhat.com/show_bug.cgi?id=1358293
--- Comment #1 from Igor Gnatenko ---
> autoreconf -ivf
move to %build
> PKG_CHECK_MODULES(ORTP, ortp >= 0.21.0,[found_ortp=true],foo=bar)
> PKG_CHECK_MODULES(MEDIASTREAMER, mediastreamer >=
> 2.8.99,[found_ms2=true],foo=bar)
add BuildRequires: pkgconfig(ortp) and pkgconfig(mediastreamer) if those are in
fedora and you think we should enable those features (don't know what those
packages do, so it's up to you to check them and think if we should enable
them)
> # Fix FSF addresses
don't do that, just report bug in upstream and note link somewhere around
License tag.
> License: GPLv2
it's GPLv2+ actually
> find %{buildroot} -name '*.la' -exec rm {} \;
find %{buildroot}%{_libdir} -type f -name '*.la' -delete -print
Other things:
* replace bcg729 with %{name} in %files section
* Add spawning test-suite in %check
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org
[Bug 1358293] Review Request: bcg729 - Opensource implementation of the G.729 codec
https://bugzilla.redhat.com/show_bug.cgi?id=1358293 Igor Gnatenko changed: What|Removed |Added Status|NEW |ASSIGNED CC||ignate...@redhat.com Assignee|nob...@fedoraproject.org|ignate...@redhat.com -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org
[Bug 1358293] Review Request: bcg729 - Opensource implementation of the G.729 codec
https://bugzilla.redhat.com/show_bug.cgi?id=1358293 Sandro Mani changed: What|Removed |Added Blocks||1357483 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1357483 [Bug 1357483] Review Request: twinkle - SIP-based VoIP client -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org
