Re: Please binNMU phonon-backend-vlc against the new libvlccore7

2013-10-31 Thread Benjamin Drung
On Mi, 2013-10-30 at 16:44 +, Adam D. Barratt wrote:
 On 2013-10-30 15:48, Benjamin Drung wrote:
  On Mi, 2013-10-30 at 13:08 +0100, Rémi Denis-Courmont wrote:
 Hello,
 
  On Wed, 30 Oct 2013 11:38:29 +, Adam D. Barratt
  a...@adam-barratt.org.uk wrote:
   On 2013-10-28 6:46, Mateusz Łukasik wrote:
   recently vlc source package has been updated and it now builds
   libvlccore7  instead of libvlccore5.
 [...]
   phonon-backend-vlc isn't the only package affected; CCing the
   maintainers - what's the plan here?
 [...]
  I concur with upstream. These packages should be fixed to use the 
  stable
  API from libvlc instead of using libvlccore. Looking at the other
  package, they all just depend on libvlc, but not on libvlccore. Just
  phonon-backend-vlc requires libvlccore.
 
 Mea culpa, I obviously had a moment and missed that only libvlccore had 
 changed SONAME (can I blame jetlag?).

No problem. You could blame whatever you want or be honest. ;)

 However, it's not just phonon-backend-vlc - goldencheetah depends on 
 both libvlc and libvlccore.

The vlc dependency is quite new (not in wheezy). May I ask you to report
a bug against goldencheetah?

-- 
Benjamin Drung
Debian  Ubuntu Developer


___
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: Please binNMU phonon-backend-vlc against the new libvlccore7

2013-10-30 Thread Adam D. Barratt

On 2013-10-28 6:46, Mateusz Łukasik wrote:

recently vlc source package has been updated and it now builds
libvlccore7  instead of libvlccore5.
It is possible to build phonon-backend-vlc against the new
libvlccore7.  Now phonon-backend-vlc have broken depends on i386 and
amd64. On others  architectures new version is not avaitable now, see
#727831.


phonon-backend-vlc isn't the only package affected; CCing the 
maintainers - what's the plan here?


# Broken Depends:
cytadela/contrib: cytadela [amd64 armel armhf i386 ia64 kfreebsd-amd64 
kfreebsd-i386 mips mipsel powerpc s390x sparc]
freetuxtv: freetuxtv [amd64 armel armhf i386 ia64 kfreebsd-amd64 
kfreebsd-i386 mips mipsel powerpc s390x sparc]
goldencheetah: goldencheetah [amd64 armel armhf i386 ia64 
kfreebsd-amd64 kfreebsd-i386 mips mipsel powerpc s390x sparc]
libplayer: libplayer2 [amd64 armel armhf i386 ia64 kfreebsd-amd64 
kfreebsd-i386 mips mipsel powerpc s390x sparc]
npapi-vlc: browser-plugin-vlc [amd64 armel armhf i386 ia64 
kfreebsd-amd64 kfreebsd-i386 mips mipsel powerpc s390x sparc]
phonon-backend-vlc: phonon-backend-vlc [amd64 armel armhf i386 ia64 
kfreebsd-amd64 kfreebsd-i386 mips mipsel powerpc s390x sparc]


I must admit to not being overjoyed at such changes being made in an 
urgency=high upload, with changelog entries such as


   * New major upstream release. (Closes: #436339, #632965, #642187,
 #698023, #593735, #724734, #665732, #700752, #704941, #708953,
 #712935, #398167, #646200, #679654, #654955, LP: #982953, #301193,
 #986785, #1038303, #1109026, #530797, #667584, #938621, #671031,
 #1080847, #1157384, #1173943)

Unless every single one of those bugs is requesting an upload of the 
new version, that changelog stanza is broken imho; it should refer to 
the actual changes made.


Regards,

Adam

___
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: Please binNMU phonon-backend-vlc against the new libvlccore7

2013-10-30 Thread Rémi Denis-Courmont
   Hello,



On Wed, 30 Oct 2013 11:38:29 +, Adam D. Barratt

a...@adam-barratt.org.uk wrote:

 On 2013-10-28 6:46, Mateusz Łukasik wrote:

 recently vlc source package has been updated and it now builds

 libvlccore7  instead of libvlccore5.

 It is possible to build phonon-backend-vlc against the new

 libvlccore7.  Now phonon-backend-vlc have broken depends on i386 and

 amd64. On others  architectures new version is not avaitable now, see

 #727831.

 

 phonon-backend-vlc isn't the only package affected; CCing the 

 maintainers - what's the plan here?



I don't know but this looks like a bug in phonon-backend-vlc to me, in

upstream and/or in Debian. The external interface to libvlc is libvlc, not

libvlccore.



libvlccore is explicitly not stable and MUST ONLY be used by VLC plugins.

In other words, unless a package installs something in

/usr/lib/vlc/plugins, it has ZERO business linking against libvlccore

directly. None of the packages below seem to pass that condition, and

definitely not phonon-backend-vlc nor browser-plugin-vlc.



 # Broken Depends:

 cytadela/contrib: cytadela [amd64 armel armhf i386 ia64 kfreebsd-amd64 

 kfreebsd-i386 mips mipsel powerpc s390x sparc]

 freetuxtv: freetuxtv [amd64 armel armhf i386 ia64 kfreebsd-amd64 

 kfreebsd-i386 mips mipsel powerpc s390x sparc]

 goldencheetah: goldencheetah [amd64 armel armhf i386 ia64 

 kfreebsd-amd64 kfreebsd-i386 mips mipsel powerpc s390x sparc]

 libplayer: libplayer2 [amd64 armel armhf i386 ia64 kfreebsd-amd64 

 kfreebsd-i386 mips mipsel powerpc s390x sparc]

 npapi-vlc: browser-plugin-vlc [amd64 armel armhf i386 ia64 

 kfreebsd-amd64 kfreebsd-i386 mips mipsel powerpc s390x sparc]

 phonon-backend-vlc: phonon-backend-vlc [amd64 armel armhf i386 ia64 

 kfreebsd-amd64 kfreebsd-i386 mips mipsel powerpc s390x sparc]

 

 I must admit to not being overjoyed at such changes being made in an 

 urgency=high upload, with changelog entries such as

 

 * New major upstream release. (Closes: #436339, #632965, #642187,

   #698023, #593735, #724734, #665732, #700752, #704941, #708953,

   #712935, #398167, #646200, #679654, #654955, LP: #982953, #301193,

   #986785, #1038303, #1109026, #530797, #667584, #938621, #671031,

   #1080847, #1157384, #1173943)

 

 Unless every single one of those bugs is requesting an upload of the 

 new version,



They do; they are all (fixed-)upstream bugs. And whomever made the upload

even *missed* a few bug numbers there.



-- 

Rémi Denis-Courmont

Sent from my collocated server

___
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: Please binNMU phonon-backend-vlc against the new libvlccore7

2013-10-30 Thread Adam D. Barratt

On 2013-10-30 12:08, Rémi Denis-Courmont wrote:


On Wed, 30 Oct 2013 11:38:29 +, Adam D. Barratt

a...@adam-barratt.org.uk wrote:

[...]
* New major upstream release. (Closes: #436339, #632965, 
#642187,



  #698023, #593735, #724734, #665732, #700752, #704941, #708953,


  #712935, #398167, #646200, #679654, #654955, LP: #982953, 
#301193,


  #986785, #1038303, #1109026, #530797, #667584, #938621, 
#671031,



  #1080847, #1157384, #1173943)





Unless every single one of those bugs is requesting an upload of the
new version,


They do; they are all (fixed-)upstream bugs. And whomever made the 
upload

even *missed* a few bug numbers there.


As a random selection, #593735 is entitled vlc: don't exclusively 
depend on ttf-freefont. That is a) a packaging change and b) not a 
request for a new upstream version.


Even if that bug were fixed upstream, that is /not/ the same as a 
request for a new upstream version to be uploaded. In this case, the 
changelog would have been better as something along the lines of:


  * New major upstream release:
- don't exclusively depend on ttf-freefont (Closes: #593735)
- change 2 (Closes: #NN)

Regards,

Adam

___
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: Please binNMU phonon-backend-vlc against the new libvlccore7

2013-10-30 Thread Benjamin Drung
On Mi, 2013-10-30 at 13:08 +0100, Rémi Denis-Courmont wrote:
Hello,

 On Wed, 30 Oct 2013 11:38:29 +, Adam D. Barratt
 a...@adam-barratt.org.uk wrote:
  On 2013-10-28 6:46, Mateusz Łukasik wrote:
  recently vlc source package has been updated and it now builds
  libvlccore7  instead of libvlccore5.
  It is possible to build phonon-backend-vlc against the new
  libvlccore7.  Now phonon-backend-vlc have broken depends on i386 and
  amd64. On others  architectures new version is not avaitable now, see
  #727831.
  
  phonon-backend-vlc isn't the only package affected; CCing the 
  maintainers - what's the plan here?

 I don't know but this looks like a bug in phonon-backend-vlc to me, in
 upstream and/or in Debian. The external interface to libvlc is libvlc, not
 libvlccore.
 
 libvlccore is explicitly not stable and MUST ONLY be used by VLC plugins.
 In other words, unless a package installs something in
 /usr/lib/vlc/plugins, it has ZERO business linking against libvlccore
 directly. None of the packages below seem to pass that condition, and
 definitely not phonon-backend-vlc nor browser-plugin-vlc.

I concur with upstream. These packages should be fixed to use the stable
API from libvlc instead of using libvlccore. Looking at the other
package, they all just depend on libvlc, but not on libvlccore. Just
phonon-backend-vlc requires libvlccore.

-- 
Benjamin Drung
Debian  Ubuntu Developer


___
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: Please binNMU phonon-backend-vlc against the new libvlccore7

2013-10-30 Thread Rémi Denis-Courmont
Le mercredi 30 octobre 2013 16:48:30 Benjamin Drung a écrit :
 These packages should be fixed to use the stable
 API from libvlc instead of using libvlccore. Looking at the other
 package, they all just depend on libvlc, but not on libvlccore. Just
 phonon-backend-vlc requires libvlccore.

Even that is a bug in upstream Phonon, AFAIK.

-- 
Rémi Denis-Courmont
http://www.remlab.net/


___
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: Please binNMU phonon-backend-vlc against the new libvlccore7

2013-10-30 Thread Adam D. Barratt

On 2013-10-30 15:48, Benjamin Drung wrote:

On Mi, 2013-10-30 at 13:08 +0100, Rémi Denis-Courmont wrote:

   Hello,

On Wed, 30 Oct 2013 11:38:29 +, Adam D. Barratt
a...@adam-barratt.org.uk wrote:
 On 2013-10-28 6:46, Mateusz Łukasik wrote:
 recently vlc source package has been updated and it now builds
 libvlccore7  instead of libvlccore5.

[...]

 phonon-backend-vlc isn't the only package affected; CCing the
 maintainers - what's the plan here?

[...]
I concur with upstream. These packages should be fixed to use the 
stable

API from libvlc instead of using libvlccore. Looking at the other
package, they all just depend on libvlc, but not on libvlccore. Just
phonon-backend-vlc requires libvlccore.


Mea culpa, I obviously had a moment and missed that only libvlccore had 
changed SONAME (can I blame jetlag?).


However, it's not just phonon-backend-vlc - goldencheetah depends on 
both libvlc and libvlccore.


Regards,

Adam

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