Bug#869414: package smplayer/16.11.0~ds0-1+deb9u1

2017-08-09 Thread Gianfranco Costamagna 

>What about #870233, sounds like a good opportunity to fix that along?


it was really too late for that one :)

Mateusz, can you please prepare an update in case you want it fixed?

G.

Bug#869414: package smplayer/16.11.0~ds0-1+deb9u1

2017-08-07 Thread Adam D. Barratt 
Control: tags -1 + pending

On Sun, 2017-08-06 at 23:26 +0200, Gianfranco Costamagna wrote:
> On Sun, 6 Aug 2017 13:40:28 +0100 Jonathan Wiltshire  wrote:
> > Control: tag -1 confirmed
> 
> uploaded.

Flagged for acceptance.

Regards,

Adam

Bug#869414: package smplayer/16.11.0~ds0-1+deb9u1

2017-08-06 Thread Moritz Mühlenhoff 
On Sun, Jul 23, 2017 at 12:20:25PM +0200, Mateusz Łukasik wrote:
> Package: release.debian.org
> Severity: normal
> Tags: stretch
> User: release.debian@packages.debian.org
> Usertags: pu
> 
> Dear SRMs,
> 
> I would like to update smplayer in Stretch to fix #869411, it was already
> fixed in unstable.

What about #870233, sounds like a good opportunity to fix that along?

Cheers,
Moritz

Bug#869414: package smplayer/16.11.0~ds0-1+deb9u1

2017-08-06 Thread Gianfranco Costamagna 
On Sun, 6 Aug 2017 13:40:28 +0100 Jonathan Wiltshire  wrote:
> Control: tag -1 confirmed

uploaded.

G.



signature.asc
Description: OpenPGP digital signature

Bug#869414: package smplayer/16.11.0~ds0-1+deb9u1

2017-08-06 Thread Jonathan Wiltshire 
Control: tag -1 confirmed

On Sun, Jul 23, 2017 at 12:20:25PM +0200, Mateusz Łukasik wrote:
> I would like to update smplayer in Stretch to fix #869411, it was already
> fixed in unstable.
> 
> The patch has been merged from upstream release.
> 
> Full source debdiff attached.

Please set the changelog distribution to "stretch", and then go ahead.

Thanks,

-- 
Jonathan Wiltshire  j...@debian.org
Debian Developer http://people.debian.org/~jmw

4096R: 0xD3524C51 / 0A55 B7C5 1223 3942 86EC  74C3 5394 479D D352 4C51

Bug#869414: package smplayer/16.11.0~ds0-1+deb9u1

2017-07-23 Thread Mateusz Łukasik 

Package: release.debian.org
Severity: normal
Tags: stretch
User: release.debian@packages.debian.org
Usertags: pu

Dear SRMs,

I would like to update smplayer in Stretch to fix #869411, it was 
already fixed in unstable.


The patch has been merged from upstream release.

Full source debdiff attached.


--
 .''`.  Mateusz Łukasik
: :' :  https://l0calh0st.pl
`. `'   Debian Member - mat...@linuxmint.pl
  `-GPG: D93B 0C12 C8D0 4D7A AFBC  FA27 CCD9 1D61 11A0 6851
diff -Nru smplayer-16.11.0~ds0/debian/changelog 
smplayer-16.11.0~ds0/debian/changelog
--- smplayer-16.11.0~ds0/debian/changelog   2016-11-22 17:31:13.0 
+0100
+++ smplayer-16.11.0~ds0/debian/changelog   2017-07-23 12:10:13.0 
+0200
@@ -1,3 +1,9 @@
+smplayer (16.11.0~ds0-1+deb9u1) stable-proposed-updates; urgency=high
+
+  * Merge from upstream fix connections to youtube. (Closes: #869411)
+
+ -- Mateusz Łukasik   Sun, 23 Jul 2017 12:10:13 +0200
+
 smplayer (16.11.0~ds0-1) unstable; urgency=medium
 
   * New upstream release.
diff -Nru smplayer-16.11.0~ds0/debian/patches/07-fixyoutube.patch 
smplayer-16.11.0~ds0/debian/patches/07-fixyoutube.patch
--- smplayer-16.11.0~ds0/debian/patches/07-fixyoutube.patch 1970-01-01 
01:00:00.0 +0100
+++ smplayer-16.11.0~ds0/debian/patches/07-fixyoutube.patch 2017-07-23 
12:09:51.0 +0200
@@ -0,0 +1,39 @@
+Description: Fix connections to youtube.
+Bug-Debian: http://bugs.debian.org/869411
+
+--- a/src/youtube/sig.cpp
 b/src/youtube/sig.cpp
+@@ -51,9 +51,11 @@
+   qDebug() << "Sig::findFunctions: sts:" << sts;
+ 
+   QString sig_name;
+-  QRegExp rx_sig("\\.sig\\|\\|([a-zA-Z0-9\\$]+)\\(");
++  //QRegExp rx_sig("\\.sig\\|\\|([a-zA-Z0-9\\$]+)\\(");
++  QRegExp rx_sig("([\"\'])signature\\1\\s*,\\s*([a-zA-Z0-9$]+)\\(");
+   if (rx_sig.indexIn(text) != -1) {
+-  sig_name = rx_sig.cap(1);
++  //sig_name = rx_sig.cap(1);
++  sig_name = rx_sig.cap(2);
+   }
+   qDebug() << "Sig::findFunctions: sig_name:" << sig_name;
+ 
+
+--- a/src/youtube/retrieveyoutubeurl.cpp
 b/src/youtube/retrieveyoutubeurl.cpp
+@@ -252,11 +252,12 @@
+   #ifdef YT_USE_SIG
+   QString html5_player;
+   #endif
+-  //QRegExp rxplayer("player-([\\d,\\w,-]+)");
+-  QRegExp rxplayer("jsbin\\/player-([\\d\\w-]+)\\/base\\.js");
++  QRegExp 
rxplayer("jsbin\\/player-([\\d\\w-]+)\\/([a-z]{2}_[A-Z]{2})\\/base\\.js");
+   if (rxplayer.indexIn(replyString) != -1) {
+-  html5_player = rxplayer.cap(1);
+-  qDebug() << "RetrieveYoutubeUrl::videoPageLoaded: html5player:" 
<< html5_player;
++  QString player = rxplayer.cap(1);
++  QString locale = rxplayer.cap(2);
++  qDebug() << "RetrieveYoutubeUrl::videoPageLoaded: html5player:" 
<< player << "locale:" << locale;
++  html5_player = player +"/"+ locale;
+   }
+ 
+   video_page = replyString;
diff -Nru smplayer-16.11.0~ds0/debian/patches/series 
smplayer-16.11.0~ds0/debian/patches/series
--- smplayer-16.11.0~ds0/debian/patches/series  2016-11-21 13:29:44.0 
+0100
+++ smplayer-16.11.0~ds0/debian/patches/series  2017-07-23 12:08:39.0 
+0200
@@ -3,3 +3,4 @@
 05-add-debian-hardening-flags.patch
 06-tryfixplaylist.patch
 01-update-mime-types.patch
+07-fixyoutube.patch