Bug 5104

Summary: Review request: glmixer - Graphic Live Mixer
Product: Package Reviews Reporter: mgansser <mgansser>
Component: Review RequestAssignee: Vasiliy Glazov <vascom2>
Status: RESOLVED EXPIRED    
Severity: enhancement CC: ferdnyc, rpmfusion-package-review, vascom2, vitaly
Priority: P1 Flags: vascom2: fedora-review?
Version: Current   
Hardware: x86_64   
OS: GNU/Linux   
URL: http://sourceforge.net/projects/glmixer/
namespace: free

Description mgansser@online.de 2018-12-08 19:02:05 CET
Spec URL: https://martinkg.fedorapeople.org/Review/SPECS/glmixer.spec
SRPM URL: https://martinkg.fedorapeople.org/Review/SRPMS/glmixer-1.7.1837-1.fc29.src.rpm

Description:Real-time video mixing software for live performance.

Fedora Account System Username: martinkg

this package belongs to rpmfusion, due x264 dependencies.

$ rpmlint -i -v glmixer.spec /home/martin/rpmbuild/SRPMS/glmixer-1.7.1837-1.fc29.src.rpm /home/martin/rpmbuild/RPMS/x86_64/glmixer-1.7.1837-1.fc29.x86_64.rpm /home/martin/rpmbuild/RPMS/x86_64/glmixer-debugsource-1.7.1837-1.fc29.x86_64.rpm /home/martin/rpmbuild/RPMS/x86_64/glmixer-debuginfo-1.7.1837-1.fc29.x86_64.rpm/home/martin/rpmbuild/RPMS/x86_64/glmixer-debuginfo-1.7.1837-1.fc29.x86_64.rpm
glmixer.spec: I: checking
glmixer.spec: W: invalid-url Source0: GLMixer-1.7.1837-Source.tar.gz
The value should be a valid, public HTTP, HTTPS, or FTP URL.

glmixer.src: I: checking
glmixer.src: I: checking-url http://sourceforge.net/projects/glmixer/ (timeout 10 seconds)
glmixer.src: W: invalid-url Source0: GLMixer-1.7.1837-Source.tar.gz
The value should be a valid, public HTTP, HTTPS, or FTP URL.

glmixer.x86_64: I: checking
glmixer.x86_64: I: checking-url http://sourceforge.net/projects/glmixer/ (timeout 10 seconds)
glmixer-debugsource.x86_64: I: checking
glmixer-debugsource.x86_64: I: checking-url http://sourceforge.net/projects/glmixer/ (timeout 10 seconds)
(none): E: no installed packages by name /home/martin/rpmbuild/RPMS/x86_64/glmixer-debuginfo-1.7.1837-1.fc29.x86_64.rpm/home/martin/rpmbuild/RPMS/x86_64/glmixer-debuginfo-1.7.1837-1.fc29.x86_64.rpm
3 packages and 1 specfiles checked; 0 errors, 2 warnings.

%changelog
* Sat Dec 08 2018 Martin Gansser <martinkg@fedoraproject.org> - 1.7.1837-1
- Initial package
Comment 1 mgansser@online.de 2018-12-08 21:33:17 CET
Spec URL: https://martinkg.fedorapeople.org/Review/SPECS/glmixer.spec
SRPM URL: https://martinkg.fedorapeople.org/Review/SRPMS/glmixer-1.7.1839-1.fc29.src.rpm

%changelog
* Sat Dec 08 2018 Martin Gansser <martinkg@fedoraproject.org> - 1.7.1839-1
- Update to 1.7.1839-1
- Add appdata.xml
Comment 2 mgansser@online.de 2018-12-11 08:27:55 CET
Spec URL: https://martinkg.fedorapeople.org/Review/SPECS/glmixer.spec
SRPM URL: https://martinkg.fedorapeople.org/Review/SRPMS/glmixer-1.7.1840-1.fc29.src.rpm


%changelog
* Mon Dec 10 2018 Martin Gansser <martinkg@fedoraproject.org> - 1.7.1840-1
- Update to 1.7.1840
- Dropped glmixer-glew-sys.patch
Comment 3 mgansser@online.de 2019-01-04 14:05:57 CET
Spec URL: https://martinkg.fedorapeople.org/Review/SPECS/glmixer.spec
SRPM URL: https://martinkg.fedorapeople.org/Review/SRPMS/glmixer-1.7.1875-1.fc29.src.rpm

%changelog
* Fri Jan 04 2019 Martin Gansser <martinkg@fedoraproject.org> - 1.7.1875-1
- Update to 1.7.1875
Comment 4 mgansser@online.de 2019-05-17 15:35:54 CEST
Spec URL: https://martinkg.fedorapeople.org/Review/SPECS/glmixer.spec
SRPM URL: https://martinkg.fedorapeople.org/Review/SRPMS/glmixer-1.7.1914-1.fc30.src.rpm


%changelog
* Fri May 17 2019 Martin Gansser <martinkg@fedoraproject.org> - 1.7.1914-1
- Update to 1.7.1914
Comment 5 Vasiliy Glazov 2019-05-17 15:51:51 CEST
I will review.
Comment 6 Vasiliy Glazov 2019-05-17 16:00:21 CEST
1. Instead
%setup -qn GLMixer-%{version}-Source
%patch0 -p1

use
%autosetup -p1 -n GLMixer-%{version}-Source

2. Add 
Requires:       hicolor-icon-theme

3. Change
%{_libdir}/%{name}/libffgl*.so

to
%{_libdir}/%{name}

4. It can be builded with Qt5?
Comment 7 mgansser@online.de 2019-05-17 16:38:20 CEST
(In reply to Vasiliy Glazov from comment #6)
> 1. Instead
> %setup -qn GLMixer-%{version}-Source
> %patch0 -p1
> 
> use
> %autosetup -p1 -n GLMixer-%{version}-Source
> 
> 2. Add 
> Requires:       hicolor-icon-theme
> 
> 3. Change
> %{_libdir}/%{name}/libffgl*.so
> 
> to
> %{_libdir}/%{name}
> 
> 4. It can be builded with Qt5?

will upload a new package, but the source code requires qt4

BUILDSTDERR: CMake Error at /usr/share/cmake/Modules/FindQt4.cmake:1314 (message):
BUILDSTDERR:   Found unsuitable Qt version "" from NOTFOUND, this code requires Qt 4.x
Comment 8 mgansser@online.de 2019-05-17 16:45:41 CEST
Spec URL: https://martinkg.fedorapeople.org/Review/SPECS/glmixer.spec
SRPM URL: https://martinkg.fedorapeople.org/Review/SRPMS/glmixer-1.7.1914-2.fc30.src.rpm


%changelog
* Fri May 17 2019 Martin Gansser <martinkg@fedoraproject.org> - 1.7.1914-2
- Use macro %%autosetup
- Add RR hicolor-icon-theme
- Use %%{_libdir}/%%{name}
Comment 9 Vitaly 2019-05-17 18:01:06 CEST
> but the source code requires qt4

Qt4 is deprecated and no longer supported by upstream. You should avoid it because it has lots of known major issues.

Please ask upstream to switch to Qt5.
Comment 10 mgansser@online.de 2019-05-17 21:11:18 CEST
(In reply to Vitaly Zaitsev from comment #9)
> > but the source code requires qt4
> 
> Qt4 is deprecated and no longer supported by upstream. You should avoid it
> because it has lots of known major issues.
> 
> Please ask upstream to switch to Qt5.


have opened upstream ticket [1]

[1] https://sourceforge.net/p/glmixer/tickets/94/
Comment 11 Nicolas Chauvet 2020-02-06 15:01:26 CET
I don't think glmixer is active upstream anymore.
Do we want to follow the review or not ? (specially as it will be still using qt4).

@Martin
I'm suggesting to close, but if you want to continue, please re-open.
Comment 12 FeRD (Frank Dana) 2020-02-06 16:14:53 CET
(In reply to Nicolas Chauvet from comment #11)
> I don't think glmixer is active upstream anymore.

?? There were 57 commits[1] to its repo in 2019, the most recent were a bunch all on 2019-11-03 and 2019-11-04.

I have no opinion on its UTILITY or whatever, and I have no idea whether they plan to modernize the code, but that doesn't strike me as a dead project.

(Though I agree that, unless they're planning to migrate to Qt5, it seems kind of pointless. And the fact that there was no response to Martin's Qt5 request ticket is a bad sign.)

[1]: https://sourceforge.net/p/glmixer/Source/commit_browser