Bug#699695: gnome-mplayer: should read and use any mplayer configuration file

[Francesco Poli]

On Sat, 9 Feb 2013 19:31:28 +0100 Francesco Poli wrote:

> On Sat, 9 Feb 2013 18:34:53 +0100 Sebastian Ramacher wrote:
[...]
> > Let's create a new bug for this. Cloned and forwarded.
> 
> Thanks!
> 
> Please let me know how the discussion with upstream develops, since I
> am having a hard time in understanding their replies (this is not the
> first time, it seems).

Hello again,
since I am not satisfied with the "workaround" suggested by upstream
and mentioned in , I am trying to see
whether I can come up with a patch that makes gnome-mplayer avoid
passing the "-volume" option to mplayer in certain situations.

When I saw the "use_volume_option" variable in src/main.c:1309,
I thought I found the way to do it: it seemed that setting that
variable to FALSE could disable the use of the "-volume" option in the
mplayer command line.

I tried to modify the src/main.c accordingly, but it seems to me that
the "use_volume_option" variable has no effect at all.
It is indeed 0 (checked with gdb), but the mplayer command line still
includes the "-volume" option...
Before this issue drives me completely crazy, could you please clarify
what's wrong in my reasoning?
Is the "use_volume_option" variable actually used for anything?
Is there an easy way to disable the passing of the "-volume" option to
mplayer (without a major code overhaul)?

Any help would be much appreciated.
Thanks for your time!


-- 
 http://www.inventati.org/frx/frx-gpg-key-transition-2010.txt
 New GnuPG key, see the transition document!
. Francesco Poli .
 GnuPG key fpr == CA01 1147 9CD2 EFDF FB82  3925 3E1C 27E1 1F69 BFFE


pgpCquJZNTZkD.pgp
Description: PGP signature
___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-multimedia-maintainers

Bug#700189: Bug#699695: gnome-mplayer: should read and use any mplayer configuration file

[Francesco Poli]

On Sat, 9 Feb 2013 20:55:10 +0100 Sebastian Ramacher wrote:

> On 2013-02-09 19:31:28, Francesco Poli wrote:
[...]
> > Please let me know how the discussion with upstream develops, since I
> > am having a hard time in understanding their replies (this is not the
> > first time, it seems).
> 
> There won't be an option for that.

That's really disappointing.

> But there is an "easy" workaround for
> the problem: Create a shell script containing something like
> 
>   #!/bin/bash
>   /usr/bin/gnome-mplayer --volume 20 "$@"
> 
> called gnome-mplayer somewhere in your $PATH before /usr/bin. Also make
> sure that software volume control is enabled.

This workaround is a kludge!

I know how to create launch scripts, but seeing upstream developers of
a graphical front-end for a video player recommending that *each* user
(with a need to easily configure the initial volume) create his/her
own launch script with a hard-coded configuration value is really
discouraging...   :-(

[...]
> 
> Since there is a workaround and there won't be any other fix for this,
> I'm closing this bug. I hope this helps.

Frankly speaking, I am not happy with the way the upstream developers
deal with bug reports coming from users.   :-(


-- 
 http://www.inventati.org/frx/frx-gpg-key-transition-2010.txt
 New GnuPG key, see the transition document!
. Francesco Poli .
 GnuPG key fpr == CA01 1147 9CD2 EFDF FB82  3925 3E1C 27E1 1F69 BFFE


pgptLxYAbGGHj.pgp
Description: PGP signature
___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-multimedia-maintainers

Bug#701039: handbrake: libhb/decavcodec.c and libhb/encavcodecaudio.c in handbrake are missing prototypes

[Paul Gevers]

Package: handbrake
Version: 0.9.8+dfsg1-2
Severity: normal
Tags: upstream

-BEGIN PGP SIGNED MESSAGE-
Hash: SHA256

As requested in e-mail conversation, I file this bug to keep track of the fact
that libhb/decavcodec.c and libhb/encavcodecaudio.c in handbrake are missing
prototypes.

Relevant parts of the e-mail conversation [1]:

>> The build on amd64 in Ubuntu is failing because handbrake seems to miss
>> a proper prototype in libhb/decavcodec.c and libhb/encavcodecaudio.c
>> (see below). This is to prevent failure on amd64 and ia64 systems.

> Matthias already approached me on IRC. The problem is understood, and
> the fix is to update the package in experimental a recent git snapshot
> that uses libavresample instead of the deprecated, and now removed
> symbols that your build is bitching about.

[1] 
http://lists.alioth.debian.org/pipermail/pkg-multimedia-maintainers/2013-February/031414.html

- -- System Information:
Debian Release: 7.0
  APT prefers testing
  APT policy: (500, 'testing'), (1, 'experimental')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 3.2.0-4-amd64 (SMP w/1 CPU core)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.12 (GNU/Linux)

iQEcBAEBCAAGBQJRJSiEAAoJEJxcmesFvXUKZrAIAKNcN/9kRv3c5oaNjKoGMLj2
HmfAS8zidoyW6NxQEpMDxNcW8OlI3vsxibGLtEdmSbpfwGNnh1L4cafVd4e2knHH
qFpXnZigxN5asmJUN+ymC4YwOu335QrDKDPwMaOPEybUCZH5AQ4Qje/zqMwihlUL
99vSPmIsvOkK9K9jns4HralvTAfXniQy/MX/eGj+bOwmBZgvJ5CIwbDWx6CuCSfK
HhUsuDBnwrGB1aF3dVFFs0Juksw5wRMj5zxrjc9C10ZsSnGAVbl+m2weWdEXp/vA
Cp/gvARee33y6lmwVh6JTigg6bBlJPGGkrmGfa4UNLSaQTuXNva//XlxqpnHjM8=
=WToI
-END PGP SIGNATURE-

___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-multimedia-maintainers

Re: handbrake 0.9.8+dfsg1-2 fails to build in Ubuntu amd64 due to missing prototype.

[Reinhard Tartler]

Hi Paul.

On Wed, Feb 20, 2013 at 8:21 PM, Paul Gevers  wrote:
> Hi Multimedia maintainers,
>
> You might have seen already (some of you responded there) that Ubuntu
> builds handbrake from experimental (on my request).

TBH, I'm not entirely convinced that this was/is good idea in the
first place. The package currently in experimental has two serious
functional limitations that handbrake users generally expect.
Unfortunately, both require libraries that use licenses that prevent
redistributors such as Ubuntu and Debian to ship them. Until we have
found and implemented a viable solution, the missing AAC encoding and
MP4 container features will seriously annoy users, to say at least.

> The build on amd64 in Ubuntu is failing because handbrake seems to miss
> a proper prototype in libhb/decavcodec.c and libhb/encavcodecaudio.c
> (see below). This is to prevent failure on amd64 and ia64 systems.

Can you please send this as a proper bug report? This is to trace
progress on this issue.

> As I don't have the knowledge yet to provide a proper fix, I rather
> thought to let you know. Do you want me to take this up with upstream,
> or can one of you provide a patch (I assume it is really simple
> actually), and help upstream with that?

Matthias already approached me on IRC. The problem is understood, and
the fix is to update the package in experimental a recent git snapshot
that uses libavresample instead of the deprecated, and now removed
symbols that your build is bitching about.

>
> See the conversation below where Daniel explains the situation.
>
> Don't hesitate to tell me how I can help here.

a. file a bug
b. clone from http://anonscm.debian.org/gitweb/?p=pkg-multimedia/handbrake.git
c. prepare the new upstream snapshot
d. publish your repo
e. inform the bug filed in step a where to fetch and review your changes



Cheers,
Reinhard



-- 
regards,
Reinhard

___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-multimedia-maintainers

handbrake 0.9.8+dfsg1-2 fails to build in Ubuntu amd64 due to missing prototype.

[Paul Gevers]

Hi Multimedia maintainers,

You might have seen already (some of you responded there) that Ubuntu
builds handbrake from experimental (on my request).

The build on amd64 in Ubuntu is failing because handbrake seems to miss
a proper prototype in libhb/decavcodec.c and libhb/encavcodecaudio.c
(see below). This is to prevent failure on amd64 and ia64 systems.

As I don't have the knowledge yet to provide a proper fix, I rather
thought to let you know. Do you want me to take this up with upstream,
or can one of you provide a patch (I assume it is really simple
actually), and help upstream with that?

See the conversation below where Daniel explains the situation.

Don't hesitate to tell me how I can help here.

Paul

On 20-02-13 12:47, Daniel Holbach wrote:
> Ok, the build log
> (https://launchpadlibrarian.net/130365583/buildlog_ubuntu-raring-amd64.handbrake_0.9.8%2Bdfsg1-2_FAILEDTOBUILD.txt.gz)
> seems to say the following:
> -
> Function `av_audio_convert_alloc' implicitly converted to pointer at
> ../libhb/decavcodec.c:1488
> Function `av_audio_convert_alloc' implicitly converted to pointer at
> ../libhb/encavcodecaudio.c:242
> 
> 
> 
> Our automated build log filter detected the problem(s) above that will
> likely cause your package to segfault on architectures where the size of
> a pointer is greater than the size of an integer, such as ia64 and amd64.
> 
> This is often due to a missing function prototype definition.
> 
> Since use of implicitly converted pointers is always fatal to the
> application
> on ia64, they are errors.  Please correct them for your next upload.
> 
> More information can be found at:
> http://wiki.debian.org/ImplicitPointerConversions
> -
> 
> That's the reason, why the build worked fine locally, but broke on the
> build daemons. :-)
> 
> Have a great day,
>  Daniel
> 




signature.asc
Description: OpenPGP digital signature
___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-multimedia-maintainers

Re: shotdetect updates

[Giulio Paci]

On 20/02/2013 17:18, Fabian Greffrath wrote:
> Am 20.02.2013 16:50, schrieb Giulio Paci:
>> I would also ask: is it possible to upload this for wheezy?
> 
> This is up to the Release Team to decide, but I guess the answer is  "no".

Ok, thank you.

>> I include the changelog here:
> 
> The only real fix is for libav 0.9 and does thus not affect wheezy. The
> rest is just "noise".

I "agree". :-)

I just noticed that I missed two entries in the changelog (I will add them):

1) it fixes a FTBFS bug in ubuntu 12.04 (thus it also does not affect
wheezy);
2) it enables hardening flags.

Does this make any difference?

In case it does not make any difference, what am I supposed to do about
these changes? Should I simply wait wheezy release and then ask again?

Bests,
Giulio.

___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-multimedia-maintainers

Re: shotdetect updates

[Fabian Greffrath]

Am 20.02.2013 16:50, schrieb Giulio Paci:

I would also ask: is it possible to upload this for wheezy?


This is up to the Release Team to decide, but I guess the answer is  "no".


I include the changelog here:


The only real fix is for libav 0.9 and does thus not affect wheezy. 
The rest is just "noise".


 - Fabian


___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-multimedia-maintainers

Re: shotdetect updates

[Giulio Paci]

Hi to all,
this is just a remainder that I am still waiting an answer about
shotdetect upload.

On 26/11/2012 18:21, Giulio Paci wrote:
>   recently I got a bug report 
> (http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=694299) about shotdetect 
> FTBFS with libav 0.9.
> A patch was attached fixing the issue, so I included it in shotdetect

> Can you upload this package again?

I would also ask: is it possible to upload this for wheezy?

I include the changelog here:

shotdetect (1.0.86-2) unstable; urgency=low

   * Add a patch to fix FTBFS with libav 0.9.
 Closes: #694299.
   * Set Debian Multimedia Maintainers as maintainer.
   * Add Giulio Paci as uploader.
   * Add Vcs-Git and Vcs-Browser fields in debian/control.
   * Fix copyright Format field.
   * Fix copyright license information for some files.

  -- Giulio Paci   Thu, 17 Jan 2013 23:08:25 +0100

Bests,
Giulio.

___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-multimedia-maintainers