[Bug 4647] New: Review Request: unifi - Ubiquiti UniFi controller
by RPM Fusion Bugzilla
https://bugzilla.rpmfusion.org/show_bug.cgi?id=4647
Bug ID: 4647
Summary: Review Request: unifi - Ubiquiti UniFi controller
Product: Package Reviews
Version: Current
Hardware: All
OS: GNU/Linux
Status: NEW
Severity: enhancement
Priority: P1
Component: Review Request
Assignee: rpmfusion-package-review(a)rpmfusion.org
Reporter: hobbes1069(a)gmail.com
CC: rpmfusion-package-review(a)rpmfusion.org
namespace: nonfree
SPEC: https://www.dropbox.com/s/dz2x7dlixt2qjju/unifi.spec
SRPM: https://www.dropbox.com/s/gfgqyl499neltph/unifi-5.5.20-2.fc26.src.rpm
Ubiquiti UniFi server is a centralized management system for UniFi suite of
devices. After the UniFi server is installed, the UniFi controller can be
accessed on any web browser. The UniFi controller allows the operator to
instantly provision thousands of UniFi devices, map out network topology,
quickly manage system traffic, and further provision individual UniFi devices.
This software can not go into the free repository as I have only been given
permission to package and redistribute. The source is not available and it also
contain pre-built binaries.
https://community.ubnt.com/t5/UniFi-Wireless/Unifi-Controller-software-li...
rpmlint output:
unifi.src: W: spelling-error Summary(en_US) Ubiquiti -> Ubiquity, Ubiquitous
unifi.src: W: name-repeated-in-summary C UniFi
unifi.src: W: spelling-error %description -l en_US Ubiquiti -> Ubiquity,
Ubiquitous
unifi.src: W: invalid-license Proprietary
unifi.src: W: invalid-url URL: http://www.ubnt.com/unifi HTTP Error 404: Not
Found
unifi.src:84: E: use-of-RPM_SOURCE_DIR
unifi.src:115: W: macro-in-comment %ghost
unifi.src:144: E: hardcoded-library-path in %{_prefix}/lib/firewalld/services
unifi.src:145: E: hardcoded-library-path in %{_prefix}/lib/firewalld/services/
unifi.src:242: E: hardcoded-library-path in
%{_prefix}/lib/firewalld/services/%{name}.xml
unifi.src: W: no-%build-section
unifi.x86_64: W: spelling-error Summary(en_US) Ubiquiti -> Ubiquity, Ubiquitous
unifi.x86_64: W: name-repeated-in-summary C UniFi
unifi.x86_64: W: spelling-error %description -l en_US Ubiquiti -> Ubiquity,
Ubiquitous
unifi.x86_64: W: invalid-license Proprietary
unifi.x86_64: W: invalid-url URL: http://www.ubnt.com/unifi HTTP Error 404: Not
Found
unifi.x86_64: E: arch-dependent-file-in-usr-share
/usr/share/unifi/lib/native/Linux/armhf/libubnt_webrtc_jni.so
unifi.x86_64: W: executable-stack
/usr/share/unifi/lib/native/Linux/armhf/libubnt_webrtc_jni.so
unifi.x86_64: E: arch-dependent-file-in-usr-share
/usr/share/unifi/lib/native/Linux/x86_64/libubnt_webrtc_jni.so
unifi.x86_64: W: non-conffile-in-etc /etc/logrotate.d/unifi
unifi.x86_64: W: dangling-symlink /usr/share/unifi/bin/mongod /usr/bin/mongod
unifi.x86_64: W: hidden-file-or-dir
/usr/share/unifi/webapps/ROOT/app-unifi/.version
unifi.x86_64: W: non-standard-uid /var/lib/unifi unifi
unifi.x86_64: W: non-standard-gid /var/lib/unifi unifi
unifi.x86_64: W: non-standard-uid /var/lib/unifi/data unifi
unifi.x86_64: W: non-standard-gid /var/lib/unifi/data unifi
unifi.x86_64: W: non-standard-uid /var/lib/unifi/data/system.properties unifi
unifi.x86_64: W: non-standard-gid /var/lib/unifi/data/system.properties unifi
unifi.x86_64: E: non-readable /var/lib/unifi/data/system.properties 0
unifi.x86_64: W: non-standard-uid /var/lib/unifi/run unifi
unifi.x86_64: W: non-standard-gid /var/lib/unifi/run unifi
unifi.x86_64: W: non-standard-uid /var/lib/unifi/work unifi
unifi.x86_64: W: non-standard-gid /var/lib/unifi/work unifi
unifi.x86_64: W: non-standard-uid /var/log/unifi unifi
unifi.x86_64: W: non-standard-gid /var/log/unifi unifi
unifi.x86_64: W: dangerous-command-in-%pre mv
unifi.x86_64: W: class-path-in-manifest /usr/share/unifi/lib/ace.jar
2 packages and 0 specfiles checked; 7 errors, 30 warnings.
--
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.
6 years, 8 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, 8 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, 8 months