[Bug 4815] New: Review request: vcmi - Heroes of Might and Magic 3
game engine
by RPM Fusion Bugzilla
https://bugzilla.rpmfusion.org/show_bug.cgi?id=4815
Bug ID: 4815
Summary: Review request: vcmi - Heroes of Might and Magic 3
game engine
Product: Package Reviews
Version: Current
Hardware: x86_64
OS: GNU/Linux
Status: NEW
Severity: enhancement
Priority: P1
Component: Review Request
Assignee: rpmfusion-package-review(a)rpmfusion.org
Reporter: mhroncok(a)redhat.com
CC: rpmfusion-package-review(a)rpmfusion.org
Blocks: 2
namespace: free
Spec URL: https://churchyard.fedorapeople.org/SRPMS/vcmi.spec
SRPM URL: https://churchyard.fedorapeople.org/SRPMS/vcmi-0.99-2.fc27.src.rpm
Description:
The purpose of VCMI project is to rewrite entire Heroes 3.5: WoG engine from
scratch, giving it new and extended possibilities. It will help to support
mods and new towns already made by fans but abandoned because of game code
limitations.
In its current state it already supports maps of any sizes, higher
resolutions and extended engine limits.
Fedora Account System Username: churchayrd
Why this package is not eligible to be included in Fedora: ffmpeg + needs bits
from original game
The output rpmlint gives on both the source and binary packages. Explain for
each message why you've chosen to ignore it.
$ rpmlint ./vcmi-0.99-2.fc27.x86_64.rpm
vcmi.x86_64: W: spelling-error %description -l en_US WoG -> Wog, Woe, Won
-- shortcut
vcmi.x86_64: W: no-manual-page-for-binary vcmibuilder
-- no man pages, oh well...
vcmi.x86_64: W: no-manual-page-for-binary vcmiclient
vcmi.x86_64: W: no-manual-page-for-binary vcmilauncher
vcmi.x86_64: W: no-manual-page-for-binary vcmiserver
1 packages and 0 specfiles checked; 0 errors, 5 warnings.
$ rpmlint ./vcmi-0.99-2.fc27.src.rpm
vcmi.src: W: spelling-error %description -l en_US WoG -> Wog, Woe, Won
-- shortcut
vcmi.src: W: invalid-url Source2: vcmi.zip
-- not downloadable directly
1 packages and 0 specfiles checked; 0 errors, 2 warnings.
This doesn't yet build on ppc64le yet, but see
https://bugzilla.redhat.com/show_bug.cgi?id=1551338
F27 scratchbuild http://koji.rpmfusion.org/koji/taskinfo?taskID=205602
Referenced Bugs:
https://bugzilla.rpmfusion.org/show_bug.cgi?id=2
[Bug 2] Tracker: New packages awaiting review
--
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.
6 years, 9 months
[Bug 4639] New: dfhack - Memory hacking library for Dwarf Fortress
and a set of tools that use it
by RPM Fusion Bugzilla
https://bugzilla.rpmfusion.org/show_bug.cgi?id=4639
Bug ID: 4639
Summary: dfhack - Memory hacking library for Dwarf Fortress and
a set of tools that use it
Product: Package Reviews
Version: Current
Hardware: x86_64
OS: GNU/Linux
Status: NEW
Severity: enhancement
Priority: P1
Component: Review Request
Assignee: rpmfusion-package-review(a)rpmfusion.org
Reporter: rosser.bjr(a)gmail.com
CC: rpmfusion-package-review(a)rpmfusion.org
namespace: nonfree
Now that Dwarf Fortress is in the distribution, I guess I should submit this
review request too.
Spec URL:
https://www.acm.jhu.edu/~bjr/fedora/dwarffortress/rpmfusion/dfhack.spec
SRPM URL:
https://www.acm.jhu.edu/~bjr/fedora/dwarffortress/rpmfusion/dfhack-0.43.0...
Description:
DFHack is a Dwarf Fortress memory access library, distributed with a wide
variety of useful scripts and plugins.
For users, it provides a significant suite of bugfixes and interface
enhancements by default, and more can be enabled. There are also many tools
(such as workflow or autodump) which can make life easier. You can even add
third-party scripts and plugins to do almost anything!
For modders, DFHack makes many things possible. Custom reactions, new
interactions, magic creature abilities, and more can be set through Scripts
for Modders and custom raws. Non-standard DFHack scripts and inits can be
stored in the raw directory, making raws or saves fully self-contained for
distribution - or for coexistence in a single DF install, even with
incompatible components.
For developers, DFHack unites the various ways tools access DF memory and
allows easier development of new tools. As an open-source project under
various copyleft licences, contributions are welcome.
FAS Username: tc01
Namespace: nonfree
Why not in Fedora: dfhack is fully open source. However it depends directly on
Dwarf Fortress, a nonfree game (nonfree/dwarffortress); dfhack communicates
directly with the running dwarffortess process. Thus it must be in nonfree.
Notes:
This package is a bit messy, which is one reason I've been hesitant to open a
review request for it. There are a number of points to note, most of which are
documented in the spec itself as well (it is quite well commented).
1. Upstream uses a lot of git submodules, but also the build fails if .git
metadata isn't present. However, when I inquired upstream, they were not
interested in releasing a "source tarball" from which DH could be built without
doing additional git checkouts. Therefore I wrote a script (see link below) to
*actually* generate the dfhack tarball for which I've been managing releases.
That is why there is no source url.
https://www.acm.jhu.edu/~bjr/fedora/dwarffortress/rpmfusion/build-dfhack-...
2. There are a number of bundled bits with no support for unbundling. Worse,
many of the bundled bits have been forked by dfhack, meaning that it's
basically never going to be possible to unbundle (the most egregious example is
likely a dfhack fork of protobuf).
3. dfhack has a plugin (and script) architecture. Most plugins are included in
dfhack itself, but some are distributed and maintained separately, somewhat
like the Linux kernel. (Note that one of the in-tree plugins, stonesense, is
distributed here as a subpackage, since it contains a fair amount of data
specific to itself). I have been meaning to figure out a better way to do
out-of-tree dfhack plugin builds, but for now I just have this package fill in
and install a pkgconfig file.
4. ...unfortunately, dfhack plugins need to link statically against a helper
library, "dfhack-version.a", and so a static devel package is needed too.
5. One of dfhack's plugins is an interpreter for ruby scripts, which works by
directly trying to dlopen "libruby.so" (the intention was to ship a bundled
libruby that would be downloaded during the build, but I upstreamed a patch to
disable the download). Currently, therefore, I've made dfhack Require
libruby-devel as a runtime dependency. While I guess I could patch this out, it
would then mean that dfhack needs to be rebuilt whenever there's a ruby soname
bump.
I think those are the major pain points with the current state of the package.
--
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.
6 years, 9 months
[Bug 4059] New: Review request: dwarftherapist - Management tool designed to run side-by-side with Dwarf Fortress
by RPM Fusion Bugzilla
https://bugzilla.rpmfusion.org/show_bug.cgi?id=4059
Bug #: 4059
Summary: Review request: dwarftherapist - Management tool
designed to run side-by-side with Dwarf Fortress
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: rosser.bjr(a)gmail.com
CC: rpmfusion-package-review(a)rpmfusion.org
Spec URL:
https://www.acm.jhu.edu/~bjr/fedora/dwarffortress/rpmfusion/dwarftherapis...
SRPM URL:
https://www.acm.jhu.edu/~bjr/fedora/dwarffortress/23/SRPMs/dwarftherapist...
Description:
Management tool designed to run side-by-side with Dwarf Fortress.
Offers several views and interface improvements to Dwarf Fortress.
Some features include:
Persistent custom professions - import and manage any number of custom
professions across all your forts.
Assign multiple dwarves to a custom profession at once to unify active labors
Manage labors and professions much more easily than in-game using a flexible
UI, allowing quick review of all dwarves at-a-glance
Display all pending changes before they're written to the game
Sort labor columns by associated skill level
Persistent and customizable display; change colors, reposition/hide
information screens
Group your dwarves by several criteria
This is a heavily modified version of the original Dwarf Therapist that
is still maintained for new versions of Dwarf Fortress.
rpmlint output:
dwarftherapist.x86_64: W: spelling-error %description -l en_US dwarves ->
wharves
dwarftherapist.x86_64: W: spelling-error %description -l en_US customizable ->
customization
dwarftherapist.x86_64: W: spelling-error %description -l en_US reposition ->
preposition, deposition, re position
dwarftherapist.x86_64: W: no-manual-page-for-binary dwarftherapist
dwarftherapist.x86_64: W: no-manual-page-for-binary DwarfTherapist
dwarftherapist.src: W: spelling-error %description -l en_US dwarves -> wharves
dwarftherapist.src: W: spelling-error %description -l en_US customizable ->
customization
dwarftherapist.src: W: spelling-error %description -l en_US reposition ->
preposition, deposition, re position
dwarftherapist.src:62: W: rpm-buildroot-usage %build %qmake_qt5
PREFIX=%{buildroot}%{_prefix}
The only questionable thing here is the rpm-buildroot-usage warning, and (for
the moment) I'm ignoring that because there doesn't seem to be another way to
change where the buildsystem installs (the DESTDIR option seems to be entirely
ignored by "make install").
Why not in Fedora:
dwarftherapist requires an installation of Dwarf Fortress on the system to be
useful. I've added a Requires: dwarffortress(x86-32), meaning that this package
would need to be in -nonfree. (The architecture dependency is needed because,
at the moment, DF is only built for 32-bit systems, although that may be
changing in the near future).
--
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.
6 years, 9 months