SBo scripts not building on current (read 1st post, pls)

bassmadrigal · 03-11-2017, 05:07 AM

Calling all mediainfo users:

Would you prefer to have a separate SlackBuild for the CLI and GUI versions even if they use the same source tarball or a combined one that does either/both? If the latter, would you prefer to have the GUI enabled by default?

To give more background, I recently inherited Ryan P.C. McQuen's *mediainfo* SlackBuilds and in the process of updating them to the latest version, I found out if you get the source through github (which provides you with a sane source name), mediainfo (cli) and mediainfo-gui programs are contained within different folders of the same source tarball. I have reworked the script to allow passing of CLI=yes|no and/or GUI=yes|no with the default being CLI=yes and GUI=no.

The scripts are all available on my github, or you can download the tar.gz of my SlackBuilds from my server.

NOTE: libzen and libmediainfo are hard dependencies and wxPython (or wxWidgets) is required if you build the GUI.

Code:

mediainfo-test (combined CLI/GUI):	github	tar.gz
mediainfo (original CLI version):	github	tar.gz
mediainfo-gui (original GUI version):	github	tar.gz

DEPENDENCIES:
libzen:		github	tar.gz
libmediainfo:	github	tar.gz
wxPython:	SBo

I appreciate any and all suggestions. Thanks!

USUARIONUEVO · 03-16-2017, 03:23 AM

MD5SUM Error ---> etherape

willysr · 03-16-2017, 04:39 AM

Quote:

Originally Posted by USUARIONUEVO

MD5SUM Error ---> etherape

Fixed in my branch

Thanks

USUARIONUEVO · 03-17-2017, 10:19 PM

Hi , the package snowballstemmer , have the md5 sum in the download link

snowballstemmer

Quote:

https://pypi.python.org/packages/20/...r-1.2.1.tar.gz#md5=643b019667a708a922172e33a99bf2fa

That , produced problems in some package managers , like "slapt-src" , because the file after download have the MD5 sun in name ...and cause errors.

The correct link , is the same, but without the "added" part of md5 sum

Quote:

https://pypi.python.org/packages/20/...r-1.2.1.tar.gz

I hope the problem exposed is understand , sorry for bad english.

USUARIONUEVO · 03-17-2017, 10:44 PM

ffmpegyag --> never compiled with ffmpeg3 branch.

This gui only compile vs 2.x ffmpeg versions.

willysr · 03-18-2017, 01:30 AM

Quote:

Originally Posted by USUARIONUEVO

Hi , the package snowballstemmer , have the md5 sum in the download link

snowballstemmer

That , produced problems in some package managers , like "slapt-src" , because the file after download have the MD5 sun in name ...and cause errors.

The correct link , is the same, but without the "added" part of md5 sum

I hope the problem exposed is understand , sorry for bad english.

Fixed in my branch

willysr · 03-18-2017, 01:33 AM

Quote:

Originally Posted by USUARIONUEVO

ffmpegyag --> never compiled with ffmpeg3 branch.

This gui only compile vs 2.x ffmpeg versions.

Have you tried the patch in AUR?
https://aur.archlinux.org/cgit/aur.g...LD?h=ffmpegyag

USUARIONUEVO · 03-18-2017, 04:55 AM

@willysr

read archlinux comments down ...

https://aur.archlinux.org/packages/ffmpegyag/

ponce · 03-18-2017, 06:40 AM

thanks for reporting the brokeness of ffmpegyag, USUARIONUEVO: as long as upstream catches up or some working patch surfaces, I'll remove it from the repository for current.

orbea · 03-18-2017, 10:16 AM

Aur has a patch embedded into their pkgbuild, maybe it might help? (I haven't tested)

Code:

prepare() {
	cd "$_short"
    patch -p2 <<EOF
diff -ruBb ../ffmpegyag_orig/src/Libav.cpp ./src/Libav.cpp
--- ../ffmpegyag_orig/src/Libav.cpp	2016-02-14 01:53:12.876667983 +0100
+++ ./src/Libav.cpp	2016-02-14 02:01:07.688583768 +0100
@@ -30,7 +30,7 @@
     // remove ffmpegyag itself from the list
     for(long f=0; f<(long)ConverterFiles.GetCount(); f++)
     {
-        if(ConverterFiles[f].IsSameAs(ConverterApplication.GetFullPath()))
+        if(ConverterFiles[f].EndsWith(wxT("ffmpegthumbnailer")) || ConverterFiles[f].IsSameAs(ConverterApplication.GetFullPath()))
         {
             ConverterFiles.RemoveAt(f);
             f--;
@@ -52,7 +52,11 @@
             // remove ffmpegyag itself from the list
             for(long f=0; f<(long)ConverterFiles.GetCount(); f++)
             {
-                if(ConverterFiles[f].AfterLast(wxFileName::GetPathSeparator()).IsSameAs(ConverterApplication.GetFullName()))
+                if(
+                        ConverterFiles[f].AfterLast(wxFileName::GetPathSeparator()).IsSameAs(ConverterApplication.GetFullName())
+                        || 
+                        ConverterFiles[f].AfterLast(wxFileName::GetPathSeparator()).EndsWith(wxT("ffmpegthumbnailer"))
+                  )
                 {
                     ConverterFiles.RemoveAt(f);
                     f--;

EOF
}

https://aur.archlinux.org/cgit/aur.g...LD?h=ffmpegyag

ponce · 03-18-2017, 10:28 AM

it's the same that willysr linked above: I tried it but compilation still breaks.

orbea · 03-18-2017, 10:32 AM

Oh sorry, I missed willysr's post...

Edit: Seems Aur and SBo may be the only places that tried to package it.

https://repology.org/metapackage/ffmpegyag/versions

USUARIONUEVO · 03-18-2017, 02:44 PM

i know the problem of ffmpegyad since ffmpeg 3.X appears ... never compile vs ffmpeg 3.x branch.

Only with ffmpeg 2.x compiled ok , but current have now ffmpeg-3 :=)

some months waiting , but development aparently is stopped.

willysr · 03-18-2017, 07:57 PM

it's all up to upstream maintainer. If they do not work with ffmpeg in current, meaning that it will be removed from SBo in the next cycle.

bassmadrigal · 03-19-2017, 12:53 PM

Looks like there's gonna be some massive editing of the -current repo to remove SDL2 libs from the REQUIRES lines.

Code:

Sun Mar 19 05:27:13 UTC 2017
l/SDL2-2.0.5-x86_64-1.txz: Added.
l/SDL2_gfx-1.0.3-x86_64-1.txz: Added.
l/SDL2_image-2.0.1-x86_64-1.txz: Added.
l/SDL2_mixer-2.0.1-x86_64-1.txz: Added.
l/SDL2_net-2.0.1-x86_64-1.txz: Added.
l/SDL2_ttf-2.0.14-x86_64-1.txz: Added.
l/ffmpeg-3.2.4-x86_64-2.txz: Rebuilt.
       The package now includes ffplay, which required the SDL2 libraries.
xap/audacious-3.8.2-x86_64-1.txz: Upgraded.
xap/audacious-plugins-3.8.2-x86_64-1.txz: Upgraded.