https://bugzilla.redhat.com/show_bug.cgi?id=1178361
Bug ID: 1178361 Summary: Review Request: stuntrally - Stunt Rally game with Track Editor, based on VDrift and OGRE Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nobody@fedoraproject.org Reporter: mgansser@alice.de QA Contact: extras-qa@fedoraproject.org CC: package-review@lists.fedoraproject.org
Spec URL: https://martinkg.fedorapeople.org/Review/SPECS/stuntrally.spec SRPM URL: https://martinkg.fedorapeople.org/Review/SRPMS/stuntrally-2.5-1.fc21.src.rpm
Description: The game features 167 tracks in 34 sceneries and 19 cars + 4 vehicles. Game modes include: Single Race, Tutorials, Championships, Challenges, Multiplayer and Split Screen. Also Replays and Ghost drive are present.
The Track Editor allows creating and modifying tracks. Both run on GNU/Linux and Windows.
The game aims at a rally style of driving (like in Richard Burns Rally), with possible stunt elements (jumps, loops, pipes) and uses a 3D spline generated road.
For full features list check our Wiki page: http://code.google.com/p/vdrift-ogre/wiki/Features
Fedora Account System Username: martinkg
%changelog * Sat Jan 03 2015 Martin Gansser martinkg@fedoraproject.org - 2.5-1 - initial build for Fedora
rpmlint stuntrally-2.5-1.fc21.x86_64.rpm stuntrally.x86_64: W: wrong-file-end-of-line-encoding /usr/share/doc/stuntrally/Readme.txt stuntrally.x86_64: W: no-manual-page-for-binary stuntrally stuntrally.x86_64: W: no-manual-page-for-binary sr-editor 1 packages and 0 specfiles checked; 0 errors, 3 warnings.
rpmlint stuntrally-2.5-1.fc21.src.rpm stuntrally.src:114: E: hardcoded-library-path in %{_prefix}/lib/OGRE/ stuntrally.src:115: E: hardcoded-library-path in %{_prefix}/lib/libshiny.a 1 packages and 0 specfiles checked; 2 errors, 0 warnings.
rpmlint stuntrally-data-2.5-1.fc21.noarch.rpm stuntrally-data.noarch: W: no-documentation 1 packages and 0 specfiles checked; 0 errors, 1 warnings.
rpmlint stuntrally-devel-2.5-1.fc21.x86_64.rpm stuntrally-devel.x86_64: W: spelling-error %description -l en_US shiny's -> shin's, shine's, shiny stuntrally-devel.x86_64: W: no-documentation 1 packages and 0 specfiles checked; 0 errors, 2 warnings.
https://bugzilla.redhat.com/show_bug.cgi?id=1178361
--- Comment #1 from MartinKG mgansser@alice.de --- (In reply to MartinKG from comment #0)
rpmlint stuntrally-2.5-1.fc21.x86_64.rpm stuntrally.x86_64: W: wrong-file-end-of-line-encoding /usr/share/doc/stuntrally/Readme.txt
could be solved by: iconv -f iso-8859-1 -t utf-8 Readme.txt > Readme.txt.utf8 ; mv Readme.txt.utf8 Readme.txt
https://bugzilla.redhat.com/show_bug.cgi?id=1178361
--- Comment #2 from MartinKG mgansser@alice.de --- upload new spec file, no SRPMS file due to size of approx. 900 MB: Spec URL: https://martinkg.fedorapeople.org/Review/stuntrally-2.5-2/stuntrally.spec
I made also a review template with: fedora-review -m fedora-rawhide-x86_64-rpmfusion_free -rn ../SRPMS/stuntrally-2.5-2.fc21.src.rpm https://martinkg.fedorapeople.org/Review/review-template/stuntrally/review.t...
%changelog * Sun Jan 04 2015 Martin Gansser martinkg@fedoraproject.org - 2.5-2 - fix wrong-script-end-of-line-encoding - added BR desktop-file-utils - added static subpackage
rpmlint output: Checking: stuntrally-2.5-2.fc22.x86_64.rpm stuntrally-devel-2.5-2.fc22.x86_64.rpm stuntrally-static-2.5-2.fc22.x86_64.rpm stuntrally-data-2.5-2.fc22.noarch.rpm stuntrally-2.5-2.fc22.src.rpm stuntrally.x86_64: W: no-manual-page-for-binary stuntrally stuntrally.x86_64: W: no-manual-page-for-binary sr-editor stuntrally-devel.x86_64: W: spelling-error %description -l en_US shiny's -> shin's, shine's, shiny stuntrally-devel.x86_64: W: no-documentation stuntrally-static.x86_64: W: no-documentation stuntrally-data.noarch: W: no-documentation stuntrally.src:132: E: hardcoded-library-path in %{_prefix}/lib/OGRE/ stuntrally.src:133: E: hardcoded-library-path in %{_prefix}/lib/libshiny.a stuntrally.src: W: invalid-url Source1: https_1://github.com/stuntrally/tracks/archive/2.5/tracks-2.5.tar.gz stuntrally.src: W: invalid-url Source0: https_1://github.com/stuntrally/stuntrally/archive/2.5/stuntrally-2.5.tar.gz 5 packages and 0 specfiles checked; 2 errors, 8 warnings.
does the fonts can remain in the package ? Generic: [ ]: Avoid bundling fonts in non-fonts packages. Note: Package contains font files
stuntrally includes the following truetype fonts:
/usr/share/stuntrally/fonts/DejaVuLGCSans.ttf /usr/share/stuntrally/fonts/LiberationSans-BoldItalic.ttf /usr/share/stuntrally/fonts/Vera.ttf /usr/share/stuntrally/fonts/VeraMono.ttf
this fonts a also available with yum for fedora: dejavu-lgc-sans-fonts.noarch : Variable-width sans-serif font faces, Latin-Greek-Cyrillic subset liberation-sans-fonts.noarch : Sans-serif fonts to replace commonly used Microsoft Arial bitstream-vera-fonts-common.noarch : Common files of the Bitstream Vera font set bitstream-vera-sans-mono-fonts.noarch : Monospace sans-serif Bitstream Vera fonts
how can we deal with this ?
https://bugzilla.redhat.com/show_bug.cgi?id=1178361
--- Comment #3 from MartinKG mgansser@alice.de --- upload new spec file, no SRPMS file due to size of approx. 900 MB: Spec URL: https://martinkg.fedorapeople.org/Review/stuntrally-2.5-3/stuntrally.spec
%changelog * Mon Jan 05 2015 Martin Gansser martinkg@fedoraproject.org - 2.5-3 - unbundle fonts - added BR dejavu-lgc-sans-fonts - added BR liberation-sans-fonts - added BR bitstream-vera-sans-fonts - added BR bitstream-vera-sans-mono-fonts
https://bugzilla.redhat.com/show_bug.cgi?id=1178361
--- Comment #4 from Antonio Trande anto.trande@gmail.com --- - You can improve building by using more cmake options. Run 'cmake -LAH' from main source directory. Use CMAKE_SKIP_RPATH:BOOL=YES too.
- DSHARE_INSTALL= is already defined in hte built-in RPM macro (see command 'rpm --eval %cmake'), but in this case must be re-defined with DSHARE_INSTALL=%{_datadir}/%{name}.
- stuntrally requires stuntrally-data; stuntrally-data requires stuntrally
It's a mutual dependency; i think you can keep only the first one.
- Why does stuntrally-static require stuntrally-devel?
- %{_prefix}/lib ???
https://bugzilla.redhat.com/show_bug.cgi?id=1178361
--- Comment #5 from MartinKG mgansser@alice.de --- (In reply to Antonio Trande from comment #4)
- You can improve building by using more cmake options. Run
'cmake -LAH' from main source directory. Use CMAKE_SKIP_RPATH:BOOL=YES too.
- DSHARE_INSTALL= is already defined in hte built-in RPM macro (see command
'rpm --eval %cmake'), but in this case must be re-defined with DSHARE_INSTALL=%{_datadir}/%{name}.
done
- stuntrally requires stuntrally-data; stuntrally-data requires stuntrally
It's a mutual dependency; i think you can keep only the first one.
done
upload new spec file, no SRPMS file due to size of approx. 900 MB: https://martinkg.fedorapeople.org/Review/stuntrally-2.5-4/stuntrally.spec
- Why does stuntrally-static require stuntrally-devel?
ok, dropped
- %{_prefix}/lib ???
the package includes two static libs:
ll /home/martin/rpmbuild/BUILDROOT/stuntrally-2.5-4.fc21.x86_64/usr/lib/ insgesamt 2100 drwxr-xr-x. 5 martin martin 4096 5. Jan 18:51 debug -rw-r--r--. 1 martin martin 2138372 5. Jan 18:51 libshiny.a drwxr-xr-x. 2 martin martin 4096 5. Jan 18:51 OGRE [martin@fc21 SPECS]$ ll /home/martin/rpmbuild/BUILDROOT/stuntrally-2.5-4.fc21.x86_64/usr/lib/OGRE/ insgesamt 416 -rw-r--r--. 1 martin martin 424970 5. Jan 18:51 libshiny.OgrePlatform.a
there exists already the Shader and material management library for OGRE as Fedora rpm package. http://rpms.famillecollet.com/rpmphp/zoom.php?rpm=shiny
how to unbundle it or is it not necessary ?
https://bugzilla.redhat.com/show_bug.cgi?id=1178361
--- Comment #6 from Antonio Trande anto.trande@gmail.com --- (In reply to MartinKG from comment #5)
- %{_prefix}/lib ???
the package includes two static libs:
ll /home/martin/rpmbuild/BUILDROOT/stuntrally-2.5-4.fc21.x86_64/usr/lib/ insgesamt 2100 drwxr-xr-x. 5 martin martin 4096 5. Jan 18:51 debug -rw-r--r--. 1 martin martin 2138372 5. Jan 18:51 libshiny.a drwxr-xr-x. 2 martin martin 4096 5. Jan 18:51 OGRE [martin@fc21 SPECS]$ ll /home/martin/rpmbuild/BUILDROOT/stuntrally-2.5-4.fc21.x86_64/usr/lib/OGRE/ insgesamt 416 -rw-r--r--. 1 martin martin 424970 5. Jan 18:51 libshiny.OgrePlatform.a
there exists already the Shader and material management library for OGRE as Fedora rpm package. http://rpms.famillecollet.com/rpmphp/zoom.php?rpm=shiny
how to unbundle it or is it not necessary ?
There are useful cmake options:
cd source/shiny cmake -LAH
Be careful to the bundled files.
https://bugzilla.redhat.com/show_bug.cgi?id=1178361
--- Comment #7 from MartinKG mgansser@alice.de --- (In reply to Antonio Trande from comment #6)
(In reply to MartinKG from comment #5)
- %{_prefix}/lib ???
the package includes two static libs:
ll /home/martin/rpmbuild/BUILDROOT/stuntrally-2.5-4.fc21.x86_64/usr/lib/ insgesamt 2100 drwxr-xr-x. 5 martin martin 4096 5. Jan 18:51 debug -rw-r--r--. 1 martin martin 2138372 5. Jan 18:51 libshiny.a drwxr-xr-x. 2 martin martin 4096 5. Jan 18:51 OGRE [martin@fc21 SPECS]$ ll /home/martin/rpmbuild/BUILDROOT/stuntrally-2.5-4.fc21.x86_64/usr/lib/OGRE/ insgesamt 416 -rw-r--r--. 1 martin martin 424970 5. Jan 18:51 libshiny.OgrePlatform.a
there exists already the Shader and material management library for OGRE as Fedora rpm package. http://rpms.famillecollet.com/rpmphp/zoom.php?rpm=shiny
how to unbundle it or is it not necessary ?
There are useful cmake options:
cd source/shiny cmake -LAH
Be careful to the bundled files.
i added -DSHINY_BUILD_OGRE_PLATFORM:BOOL=NO to the cmake build section. but the Linking CXX executable ../stuntrally part fails with the error message:
/usr/bin/ld: cannot find -lshiny.OgrePlatform collect2: error: ld returned 1 exit status source/CMakeFiles/stuntrally.dir/build.make:3651: recipe for target 'stuntrally' failed make[2]: *** [stuntrally] Error 1 make[2]: Leaving directory '/home/martin/rpmbuild/BUILD/stuntrally-2.5/build' CMakeFiles/Makefile2:147: recipe for target 'source/CMakeFiles/stuntrally.dir/all' failed make[1]: *** [source/CMakeFiles/stuntrally.dir/all] Error 2 make[1]: Leaving directory '/home/martin/rpmbuild/BUILD/stuntrally-2.5/build' Makefile:120: recipe for target 'all' failed
the library exists at this location: /usr/lib64/OGRE/libshiny.OgrePlatform.so
how do i add this lib path to the build flags ?
https://bugzilla.redhat.com/show_bug.cgi?id=1178361
--- Comment #8 from MartinKG mgansser@alice.de --- i added also this linker flag, stuntrally now compiles, but libshiny.a is still there.
-DCMAKE_EXE_LINKER_FLAGS="-L%{_libdir}/OGRE -lshiny.OgrePlatform" \
Error: Installed (but not packed) file (s) found: /usr/lib/libshiny.a
Spec URL: https://martinkg.fedorapeople.org/Review/stuntrally-2.5-4/stuntrally.spec
https://bugzilla.redhat.com/show_bug.cgi?id=1178361
--- Comment #9 from Igor Gnatenko i.gnatenko.brain@gmail.com --- shiny-devel: /usr/share/doc/shiny-devel/FindSHINY.cmake
you can use it for find shiny.
/usr/lib/libshiny.a: you want remove all bundles. I did some patches when tried to package stuntrally. It's hard.
https://fedorapeople.org/cgit/ignatenkobrain/public_git/stuntrally.git/ https://github.com/ignatenkobrain/stuntrally/tree/fedora-2.3
There some patches to unbundle some libs.
This is hard work and if you have time and wat to get this package in Fedora - I can help. ping me via mail to get help ;)
https://bugzilla.redhat.com/show_bug.cgi?id=1178361
Igor Gnatenko i.gnatenko.brain@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED Assignee|nobody@fedoraproject.org |i.gnatenko.brain@gmail.com
https://bugzilla.redhat.com/show_bug.cgi?id=1178361
--- Comment #10 from MartinKG mgansser@alice.de --- (In reply to Igor Gnatenko from comment #9)
shiny-devel: /usr/share/doc/shiny-devel/FindSHINY.cmake
you can use it for find shiny.
i know this, i already placed FindSHINY.cmake in the folder stuntrally-2.5/cmake but i think there is more work needed ...
/usr/lib/libshiny.a: you want remove all bundles. I did some patches when tried to package stuntrally. It's hard.
without unbundle the libraries, stuntrally cannot get into the Fedora repos.
https://fedorapeople.org/cgit/ignatenkobrain/public_git/stuntrally.git/ https://github.com/ignatenkobrain/stuntrally/tree/fedora-2.3
There some patches to unbundle some libs.
This is hard work and if you have time and want to get this package in Fedora
- I can help. ping me via mail to get help ;)
i will try it and will contact you.
https://bugzilla.redhat.com/show_bug.cgi?id=1178361
--- Comment #11 from MartinKG mgansser@alice.de --- Created attachment 978352 --> https://bugzilla.redhat.com/attachment.cgi?id=978352&action=edit 0001-unbundle-tinyxml.patch
https://bugzilla.redhat.com/show_bug.cgi?id=1178361
--- Comment #12 from MartinKG mgansser@alice.de --- Created attachment 978353 --> https://bugzilla.redhat.com/attachment.cgi?id=978353&action=edit 0002-unbundle-bullet.patch
https://bugzilla.redhat.com/show_bug.cgi?id=1178361
--- Comment #13 from MartinKG mgansser@alice.de --- Created attachment 978354 --> https://bugzilla.redhat.com/attachment.cgi?id=978354&action=edit 0003-unbundle-shiny.patch
https://bugzilla.redhat.com/show_bug.cgi?id=1178361
--- Comment #14 from MartinKG mgansser@alice.de --- Created attachment 978356 --> https://bugzilla.redhat.com/attachment.cgi?id=978356&action=edit 0004-unbundle-pagedgeometry.patch
https://bugzilla.redhat.com/show_bug.cgi?id=1178361
--- Comment #15 from MartinKG mgansser@alice.de --- i attached the patches for unbundle tinyxml, bullet, shiny and pagedgeometry here again, because they differs from Igor's patches on https://fedorapeople.org/cgit/ignatenkobrain/public_git/stuntrally.git/tree/
82 % of the code compiles fine with the recent spec file
https://martinkg.fedorapeople.org/Review/stuntrally/SPECS/stuntrally-2.5-5/s...
but then a error message comes:
/usr/bin/cmake -E cmake_progress_report /home/martin/rpmbuild/BUILD/stuntrally-2.5/build/CMakeFiles [ 82%] Building CXX object source/CMakeFiles/stuntrally.dir/ogre/common/Slider.cpp.o cd /home/martin/rpmbuild/BUILD/stuntrally-2.5/build/source && /usr/bin/c++ -DOGRE_PLUGIN_DIR_DBG="/lib64/OGRE" -DOGRE_PLUGIN_DIR_REL="/lib64/OGRE" -DSHARED_DATA_DIR="/usr/share/stuntrally" -O2 -g -pipe -Wall -Werror=format-security -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector-strong --param=ssp-buffer-size=4 -grecord-gcc-switches -m64 -mtune=generic -fno-strict-aliasing -fPIC -O3 -DNDEBUG -I/usr/include/OGRE -I/usr/include/OGRE/Terrain -I/usr/include/OGRE/Paging -I/usr/include/OGRE/Overlay -I/usr/include/bullet -I/usr/include/SDL2 -I/usr/include/MYGUI -I/home/martin/rpmbuild/BUILD/stuntrally-2.5/source/btOgre -I/home/martin/rpmbuild/BUILD/stuntrally-2.5/source/ogre -I/home/martin/rpmbuild/BUILD/stuntrally-2.5/source/ogre/common -I/home/martin/rpmbuild/BUILD/stuntrally-2.5/source/paged-geom -I/home/martin/rpmbuild/BUILD/stuntrally-2.5/source/road -I/home/martin/rpmbuild/BUILD/stuntrally-2.5/source/vdrift -o CMakeFiles/stuntrally.dir/ogre/common/Slider.cpp.o -c /home/martin/rpmbuild/BUILD/stuntrally-2.5/source/ogre/common/Slider.cpp In file included from /home/martin/rpmbuild/BUILD/stuntrally-2.5/source/ogre/common/Slider.cpp:3:0: /home/martin/rpmbuild/BUILD/stuntrally-2.5/source/ogre/common/Slider.h: In constructor 'MyGUI::Slider::Slider()': /home/martin/rpmbuild/BUILD/stuntrally-2.5/source/ogre/common/Slider.h:56:9: warning: 'MyGUI::Slider::mfValue' will be initialized after [-Wreorder] float mfValue; ^ /home/martin/rpmbuild/BUILD/stuntrally-2.5/source/ogre/common/Slider.h:24:9: warning: 'float MyGUI::Slider::mfDefault' [-Wreorder] float mfDefault; ^ /home/martin/rpmbuild/BUILD/stuntrally-2.5/source/ogre/common/Slider.cpp:12:2: warning: when initialized here [-Wreorder] Slider::Slider() : ^ /home/martin/rpmbuild/BUILD/stuntrally-2.5/source/ogre/common/SceneTrees.cpp: In member function 'void CScene::CreateTrees()': /home/martin/rpmbuild/BUILD/stuntrally-2.5/source/ogre/common/SceneTrees.cpp:194:38: error: 'class Forests::PagedGeometry' has no member named 'getTempDir' resMgr.addResourceLocation(trees->getTempDir(), "FileSystem", "BinFolder"); ^ that means that OGRE with pagedgeometry must be unbundled also ?
https://bugzilla.redhat.com/show_bug.cgi?id=1178361
MartinKG mgansser@alice.de changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |mikhirev@gmail.com
--- Comment #16 from MartinKG mgansser@alice.de --- NEWS:
- Development stopped (ended). [1] - Update to 2.6 - Bullet was unbundled - new rpm spec file [2] - new 0001-stuntrally-2.6-unbundle-tinyxml.patch [3] - new 0002-stuntrally-2.6-unbundle-shiny.patch [4] - new 0003-stuntrally-2.6-unbundle-pagedgeometry.patch [5]
[1] http://forum.freegamedev.net/viewtopic.php?f=81&t=6563&sid=446bf3fcd...
[2] https://martinkg.fedorapeople.org/Review/stuntrally/stuntrally-2.6-1/SPECS/s... [3] https://martinkg.fedorapeople.org/Review/stuntrally/stuntrally-2.6-1/Patches... [4] https://martinkg.fedorapeople.org/Review/stuntrally/stuntrally-2.6-1/Patches... [5] https://martinkg.fedorapeople.org/Review/stuntrally/stuntrally-2.6-1/Patches...
but the compilations always fails with the pagedgeometry.patch on the same place with this error:
/home/martin/rpmbuild/BUILD/stuntrally-2.6/source/ogre/common/SceneTrees.cpp: In member function 'void CScene::CreateTrees()': /home/martin/rpmbuild/BUILD/stuntrally-2.6/source/ogre/common/SceneTrees.cpp:203:38: error: 'class Forests::PagedGeometry' has no member named 'getTempDir' resMgr.addResourceLocation(trees->getTempDir(), "FileSystem", "BinFolder");
https://bugzilla.redhat.com/show_bug.cgi?id=1178361
MartinKG mgansser@alice.de changed:
What |Removed |Added ---------------------------------------------------------------------------- Depends On| |1298316
Referenced Bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=1298316 [Bug 1298316] error: 'class Forests::PagedGeometry' has no member named 'getTempDir'
https://bugzilla.redhat.com/show_bug.cgi?id=1178361
Ben Rosser rosser.bjr@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks| |1364745
Referenced Bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=1364745 [Bug 1364745] (FE-GAMESIG) Games SIG tracker bug
https://bugzilla.redhat.com/show_bug.cgi?id=1178361
Igor Gnatenko ignatenko@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution|--- |WONTFIX Last Closed| |2017-03-11 14:34:41
--- Comment #17 from Igor Gnatenko ignatenko@redhat.com --- Sorry, I don't have time for this =(
Feel free to reopen new review request.
https://bugzilla.redhat.com/show_bug.cgi?id=1178361
Igor Gnatenko ignatenko@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|CLOSED |NEW Resolution|WONTFIX |--- Assignee|ignatenko@redhat.com |nobody@fedoraproject.org Keywords| |Reopened
--- Comment #18 from Igor Gnatenko ignatenko@redhat.com --- whoa.., I was not reporter, so sorry =(
https://bugzilla.redhat.com/show_bug.cgi?id=1178361 Bug 1178361 depends on bug 1298316, which changed state.
Bug 1298316 Summary: error: 'class Forests::PagedGeometry' has no member named 'getTempDir' https://bugzilla.redhat.com/show_bug.cgi?id=1298316
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |CLOSED Resolution|--- |EOL
https://bugzilla.redhat.com/show_bug.cgi?id=1178361
Ben Rosser rosser.bjr@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |mgansser@alice.de, | |rosser.bjr@gmail.com Flags| |needinfo?(mgansser@alice.de | |)
--- Comment #19 from Ben Rosser rosser.bjr@gmail.com --- Hi Martin,
Are you still interested in this submission?
It's not clear to me what the final status was-- does the package currently not build?
https://bugzilla.redhat.com/show_bug.cgi?id=1178361
Ben Rosser rosser.bjr@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |CLOSED Blocks| |201449 (FE-DEADREVIEW) Resolution|--- |NOTABUG Last Closed|2017-03-11 14:34:41 |2018-11-01 12:53:43
--- Comment #20 from Ben Rosser rosser.bjr@gmail.com --- Hi Martin,
It's been (much) more than a week since my last request for update-- if you're still interested in this review feel free to re-open the ticket! Otherwise, I'm going to close this in accordance with the stalled reviews policy.
https://fedoraproject.org/wiki/Policy_for_stalled_package_reviews
Referenced Bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=201449 [Bug 201449] FE-DEADREVIEW -- Reviews stalled due to lack of submitter response should be blocking this bug.
package-review@lists.fedoraproject.org