1:15 p.m.
https://bugzilla.rpmfusion.org/show_bug.cgi?id=4214
Bug #: 4214
Summary: Review Rewuqst: simplescreenrecorder - screen recorder
for Linux
Classification: Unclassified
Product: Package Reviews
Version: Current
Platform: All
OS/Version: GNU/Linux
Status: NEW
Severity: normal
Priority: P5
Component: Review Request
AssignedTo: rpmfusion-package-review(a)rpmfusion.org
ReportedBy: vascom2(a)gmail.com
CC: rpmfusion-package-review(a)rpmfusion.org
URL: http://www.maartenbaert.be/simplescreenrecorder
Description:
SimpleScreenRecorder is a screen recorder for Linux.
Despite the name, this program is actually quite complex.
It's 'simple' in the sense that it's easier to use than ffmpeg/avconv or
VLC.
SPEC:
https://raw.githubusercontent.com/RussianFedora/simplescreenrecorder/mast...
SRPM:
http://koji.russianfedora.pro/kojifiles/work/tasks/7018/87018/simplescree...
--
Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
You are the assignee for the bug.
1:16 p.m.
New subject: [Bug 4214] Review Rewuqst: simplescreenrecorder - screen recorder for Linux
https://bugzilla.rpmfusion.org/show_bug.cgi?id=4214
Vasiliy Glazov <vascom2(a)gmail.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |fast.rizwaan(a)gmail.com
--- Comment #1 from Vasiliy Glazov <vascom2(a)gmail.com> 2016-08-26 15:16:17 CEST ---
*** Bug 3048 has been marked as a duplicate of this bug. ***
--
Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
You are the assignee for the bug.
1:18 p.m.
New subject: [Bug 4214] Review Request: simplescreenrecorder - screen recorder for Linux
https://bugzilla.rpmfusion.org/show_bug.cgi?id=4214
Andrea Musuruane <musuruan(a)gmail.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |musuruan(a)gmail.com
Summary|Review Rewuqst: |Review Request:
|simplescreenrecorder - |simplescreenrecorder -
|screen recorder for Linux |screen recorder for Linux
--
Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
You are the assignee for the bug.
1:45 p.m.
New subject: [Bug 4214] Review Request: simplescreenrecorder - screen recorder for Linux
https://bugzilla.rpmfusion.org/show_bug.cgi?id=4214
MartinKG <mgansser(a)alice.de> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |mgansser(a)alice.de
--- Comment #2 from MartinKG <mgansser(a)alice.de> 2016-08-26 15:45:57 CEST ---
[!]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
beginning of %install.
Delete the rm -rf $BUILDROOT in %install
--
Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
You are the assignee for the bug.
7:21 p.m.
New subject: [Bug 4214] Review Request: simplescreenrecorder - screen recorder for Linux
https://bugzilla.rpmfusion.org/show_bug.cgi?id=4214
Sérgio Basto <sergio(a)serjux.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |sergio(a)serjux.com
Blocks| |3
--- Comment #3 from Sérgio Basto <sergio(a)serjux.com> 2016-08-26 21:21:49 CEST ---
I also would approve it ;)
--
Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
You are the assignee for the bug.
7:28 p.m.
New subject: [Bug 4214] Review Request: simplescreenrecorder - screen recorder for Linux
https://bugzilla.rpmfusion.org/show_bug.cgi?id=4214
Antonio Trande <anto.trande(a)gmail.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |anto.trande(a)gmail.com
--- Comment #4 from Antonio Trande <anto.trande(a)gmail.com> 2016-08-26 21:28:21 CEST
---
Since libssr-glinject.so is a private library, it can be provided by main
package.
In my opinion, 'simplescreenrecorder-libs' package can be removed.
--
Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
You are the assignee for the bug.
5:44 a.m.
New subject: [Bug 4214] Review Request: simplescreenrecorder - screen recorder for Linux
https://bugzilla.rpmfusion.org/show_bug.cgi?id=4214
--- Comment #5 from Vasiliy Glazov <vascom2(a)gmail.com> 2016-08-27 07:44:55 CEST ---
(In reply to comment #3)
I also would approve it ;)
As you can see
https://raw.githubusercontent.com/RussianFedora/simplescreenrecorder/mast...
%install
rm -rf %{buildroot}
is already present.
--
Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
You are the assignee for the bug.
5:59 a.m.
New subject: [Bug 4214] Review Request: simplescreenrecorder - screen recorder for Linux
https://bugzilla.rpmfusion.org/show_bug.cgi?id=4214
--- Comment #6 from Vasiliy Glazov <vascom2(a)gmail.com> 2016-08-27 07:59:00 CEST ---
(In reply to comment #4)
Since libssr-glinject.so is a private library, it can be provided by
main
package.
In my opinion, 'simplescreenrecorder-libs' package can be removed.
If it is not so important I prefer keep simplescreenrecorder-libs subpackage.
Because simplescreenrecorder is already present in russianfedora repo and
update to new package will cause error during update:
Error: Transaction check error:
file /usr/lib64/simplescreenrecorder/libssr-glinject.so from install of
simplescreenrecorder-0.3.6-4.fc24.R.x86_64 conflicts with file from package
simplescreenrecorder-libs-0.3.6-3.fc24.R.x86_64
--
Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
You are the assignee for the bug.
7:30 a.m.
New subject: [Bug 4214] Review Request: simplescreenrecorder - screen recorder for Linux
https://bugzilla.rpmfusion.org/show_bug.cgi?id=4214
--- Comment #7 from Andrea Musuruane <musuruan(a)gmail.com> 2016-08-27 09:30:44 CEST
---
(In reply to comment #5)
(In reply to comment #3)
> I also would approve it ;)
As you can see
https://raw.githubusercontent.com/RussianFedora/simplescreenrecorder/mast...
%install
rm -rf %{buildroot}
is already present.
It is not necessary to remove the contents of the buildroot as the first line
of %install.
https://fedoraproject.org/wiki/Packaging:Guidelines#Tags_and_Sections
--
Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
You are the assignee for the bug.
9:12 a.m.
New subject: [Bug 4214] Review Request: simplescreenrecorder - screen recorder for Linux
https://bugzilla.rpmfusion.org/show_bug.cgi?id=4214
leigh scott <leigh123linux(a)gmail.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |leigh123linux(a)gmail.com
--- Comment #8 from leigh scott <leigh123linux(a)gmail.com> 2016-08-27 11:12:36 CEST
---
(In reply to comment #6)
(In reply to comment #4)
> Since libssr-glinject.so is a private library, it can be provided by main
> package.
> In my opinion, 'simplescreenrecorder-libs' package can be removed.
If it is not so important I prefer keep simplescreenrecorder-libs subpackage.
Because simplescreenrecorder is already present in russianfedora repo and
update to new package will cause error during update:
You could use obsoletes and provides to fix the update error.
Error: Transaction check error:
file /usr/lib64/simplescreenrecorder/libssr-glinject.so from install of
simplescreenrecorder-0.3.6-4.fc24.R.x86_64 conflicts with file from package
simplescreenrecorder-libs-0.3.6-3.fc24.R.x86_64
--
Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
You are the assignee for the bug.
9:26 a.m.
New subject: [Bug 4214] Review Request: simplescreenrecorder - screen recorder for Linux
https://bugzilla.rpmfusion.org/show_bug.cgi?id=4214
--- Comment #9 from Antonio Trande <anto.trande(a)gmail.com> 2016-08-27 11:26:16 CEST
---
(In reply to comment #6)
(In reply to comment #4)
> Since libssr-glinject.so is a private library, it can be provided by main
> package.
> In my opinion, 'simplescreenrecorder-libs' package can be removed.
If it is not so important I prefer keep simplescreenrecorder-libs subpackage.
Because simplescreenrecorder is already present in russianfedora repo and
update to new package will cause error during update:
Error: Transaction check error:
file /usr/lib64/simplescreenrecorder/libssr-glinject.so from install of
simplescreenrecorder-0.3.6-4.fc24.R.x86_64 conflicts with file from package
simplescreenrecorder-libs-0.3.6-3.fc24.R.x86_64
All third party repos shold be disabled; enable them one-by-one when needed.
Even if there is no any rule that prohibits to split private libraries in a
separated sub-package, in this case it is a file required by
'simplescreenrecorder' only, whereas 'libssr-glinject.so' is totally
useless as
stand-alone file.
--
Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
You are the assignee for the bug.
5:09 p.m.
New subject: [Bug 4214] Review Request: simplescreenrecorder - screen recorder for Linux
https://bugzilla.rpmfusion.org/show_bug.cgi?id=4214
--- Comment #10 from Sérgio Basto <sergio(a)serjux.com> 2016-08-27 19:09:53 CEST ---
(In reply to comment #7)
(In reply to comment #5)
> (In reply to comment #3)
> > I also would approve it ;)
>
> As you can see
>
https://raw.githubusercontent.com/RussianFedora/simplescreenrecorder/mast...
>
> %install
> rm -rf %{buildroot}
>
> is already present.
It is not necessary to remove the contents of the buildroot as the first line
of %install.
https://fedoraproject.org/wiki/Packaging:Guidelines#Tags_and_Sections
yes , please delete the rm -rf $BUILDROOT in %install
--
Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
You are the assignee for the bug.
3009
days inactive
3010
days old
rpmfusion-developers@lists.rpmfusion.org
11 comments
1 participants
participants (1)
-
RPM Fusion Bugzilla