https://bugzilla.redhat.com/show_bug.cgi?id=1474033
--- Comment #5 from Andrey Maslennikov ---
Koji build: https://koji.fedoraproject.org/koji/taskinfo?taskID=21104781
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this pro
https://bugzilla.redhat.com/show_bug.cgi?id=1479022
Ye Cheng <18969068...@163.com> changed:
What|Removed |Added
CC||18969068...@163.com
-
https://bugzilla.redhat.com/show_bug.cgi?id=1336159
Igor Gnatenko changed:
What|Removed |Added
Status|POST|CLOSED
Resolution|---
https://bugzilla.redhat.com/show_bug.cgi?id=1336168
Bug 1336168 depends on bug 1336159, which changed state.
Bug 1336159 Summary: Review Request: golang-github-rubyist-tracerx - Output
tracing information in your Go app based on environment variables
https://bugzilla.redhat.com/show_bug.cgi?id=13
https://bugzilla.redhat.com/show_bug.cgi?id=1336161
--- Comment #3 from Igor Gnatenko ---
(In reply to clime from comment #2)
> Can this be pushed? Or do you want to hand it over, Igor?
Please, someone take over this ;)
--
You are receiving this mail because:
You are on the CC list for the bu
https://bugzilla.redhat.com/show_bug.cgi?id=1479280
Bug ID: 1479280
Summary: Review Request:python-jieba - Chinese Words
Segementation Utilities
Product: Fedora
Version: rawhide
Component: Package Review
Severity:
https://bugzilla.redhat.com/show_bug.cgi?id=1479280
Ye Cheng <18969068...@163.com> changed:
What|Removed |Added
Blocks||177841 (FE-NEEDSPONSOR)
https://bugzilla.redhat.com/show_bug.cgi?id=1479027
Fabio Valentini changed:
What|Removed |Added
CC||decatho...@gmail.com
Assign
https://bugzilla.redhat.com/show_bug.cgi?id=1479022
Ondřej Lysoněk changed:
What|Removed |Added
Status|NEW |ASSIGNED
CC|
https://bugzilla.redhat.com/show_bug.cgi?id=1434578
--- Comment #9 from Kalev Lember ---
Thanks! I've updated it to 2.0.1.
The gcc BR is not strictly necessary as it's currently included in the default
buildroot, but the packaging guidelines say that this can change over time and
we should not
https://bugzilla.redhat.com/show_bug.cgi?id=1469767
František Zatloukal changed:
What|Removed |Added
CC||fzatl...@redhat.com
--- Comment
https://bugzilla.redhat.com/show_bug.cgi?id=1479022
--- Comment #2 from Jaroslav Škarvada ---
(In reply to Ye Cheng from comment #1)
Thanks, I am going to address the problems in next version.
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified
https://bugzilla.redhat.com/show_bug.cgi?id=1341662
František Zatloukal changed:
What|Removed |Added
Depends On||1469767
Referenced Bugs:
http
https://bugzilla.redhat.com/show_bug.cgi?id=1469767
František Zatloukal changed:
What|Removed |Added
Blocks||1341662
Referenced Bugs:
http
https://bugzilla.redhat.com/show_bug.cgi?id=1479022
--- Comment #3 from Jaroslav Škarvada ---
(In reply to Ye Cheng from comment #1)
> (In reply to Jaroslav Škarvada from comment #0)
>
> This is an unofficial review.
>
> Issues:
> ===
> - All build dependencies are listed in BuildRequires
https://bugzilla.redhat.com/show_bug.cgi?id=1344276
Niels de Vos changed:
What|Removed |Added
Status|ASSIGNED|NEW
Assignee|ktdre...@ktdreyer
https://bugzilla.redhat.com/show_bug.cgi?id=1466961
José Matos changed:
What|Removed |Added
Flags|fedora-review? |fedora-review+
--- Comment #14 from José
https://bugzilla.redhat.com/show_bug.cgi?id=1476589
--- Comment #4 from sensor@gmail.com ---
Diff:
https://github.com/FZUG/repo/commit/beffb7e08c0dd0cb99167dd24a544a33932cb070
> This will fail the mock build because network access is not available at
> build time and these downloaded libr
https://bugzilla.redhat.com/show_bug.cgi?id=1466961
--- Comment #15 from José Matos ---
Now that the package is approved the author suggests that a new release (1.2)
will follow soon. :-)
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about
https://bugzilla.redhat.com/show_bug.cgi?id=1434578
--- Comment #10 from c72...@yahoo.de ---
Package Review
==
Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed
Issues:
===
- All build dependencies are listed in BuildRequires,
https://bugzilla.redhat.com/show_bug.cgi?id=1434578
c72...@yahoo.de changed:
What|Removed |Added
Flags|fedora-review? |fedora-review+
--- Comment #11 from
https://bugzilla.redhat.com/show_bug.cgi?id=1434578
--- Comment #12 from Kalev Lember ---
Excellent, thanks for the review!
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_
https://bugzilla.redhat.com/show_bug.cgi?id=1198760
Bug 1198760 depends on bug 1249010, which changed state.
Bug 1249010 Summary: the package nodejs-js-beautify doesn't pack the python
library from the same source code
https://bugzilla.redhat.com/show_bug.cgi?id=1249010
What|Remov
https://bugzilla.redhat.com/show_bug.cgi?id=1427634
--- Comment #20 from Fabio Valentini ---
Updated .spec and SRPM files for the newest 0.14.34 upstream release:
Spec URL: https://decathorpe.fedorapeople.org/packages/syncthing.spec
SRPM URL:
https://decathorpe.fedorapeople.org/packages/synct
https://bugzilla.redhat.com/show_bug.cgi?id=1476589
--- Comment #5 from sensor@gmail.com ---
SPEC:
https://copr-be.cloud.fedoraproject.org/results/mosquito/deepin/fedora-25-x86_64/00587960-deepin-api/deepin-api.spec
SRPM:
https://copr-be.cloud.fedoraproject.org/results/mosquito/deepin/fedo
https://bugzilla.redhat.com/show_bug.cgi?id=1228942
Bug 1228942 depends on bug 1249010, which changed state.
Bug 1249010 Summary: the package nodejs-js-beautify doesn't pack the python
library from the same source code
https://bugzilla.redhat.com/show_bug.cgi?id=1249010
What|Remov
https://bugzilla.redhat.com/show_bug.cgi?id=1467054
Vít Ondruch changed:
What|Removed |Added
Flags|fedora-review? |fedora-review+
--- Comment #5 from Vít
https://bugzilla.redhat.com/show_bug.cgi?id=1279579
Bug 1279579 depends on bug 1291010, which changed state.
Bug 1291010 Summary: [ARM] terminate called after throwing an instance of
'std::length_error'
https://bugzilla.redhat.com/show_bug.cgi?id=1291010
What|Removed
https://bugzilla.redhat.com/show_bug.cgi?id=1285112
Bug 1285112 depends on bug 1291010, which changed state.
Bug 1291010 Summary: [ARM] terminate called after throwing an instance of
'std::length_error'
https://bugzilla.redhat.com/show_bug.cgi?id=1291010
What|Removed
https://bugzilla.redhat.com/show_bug.cgi?id=1279176
Bug 1279176 depends on bug 1291010, which changed state.
Bug 1291010 Summary: [ARM] terminate called after throwing an instance of
'std::length_error'
https://bugzilla.redhat.com/show_bug.cgi?id=1291010
What|Removed
https://bugzilla.redhat.com/show_bug.cgi?id=1288739
Bug 1288739 depends on bug 1291010, which changed state.
Bug 1291010 Summary: [ARM] terminate called after throwing an instance of
'std::length_error'
https://bugzilla.redhat.com/show_bug.cgi?id=1291010
What|Removed
https://bugzilla.redhat.com/show_bug.cgi?id=1478052
Vít Ondruch changed:
What|Removed |Added
Status|NEW |ASSIGNED
Assignee|nob...@fedora
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
https://bugzilla.redhat.com/show_bug.cgi?id=1478052
Vít Ondruch changed:
What|Removed |Added
Flags|fedora-review? |fedora-review+
--- Comment #2 from Vít
https://bugzilla.redhat.com/show_bug.cgi?id=1434746
--- Comment #1 from Lukas Berk ---
Updated SPEC: https://bintray.com/pcp/f26/download_file?file_path=uom-se.spec
Updated SRPM:
https://bintray.com/pcp/f26/download_file?file_path=uom-se-1.0.4-3.fc26.src.rpm
--
You are receiving this mail bec
https://bugzilla.redhat.com/show_bug.cgi?id=1434746
Lukas Berk changed:
What|Removed |Added
Flags|fedora-review? |fedora-review+
--- Comment #2 from Lukas
https://bugzilla.redhat.com/show_bug.cgi?id=1427634
--- Comment #21 from Fabio Valentini ---
Updated .spec and SRPM files for the newest hotfix 0.14.35 upstream release:
Spec URL: https://decathorpe.fedorapeople.org/packages/syncthing.spec
SRPM URL:
https://decathorpe.fedorapeople.org/package
https://bugzilla.redhat.com/show_bug.cgi?id=1464194
--- Comment #7 from Rutvij Karkhanis ---
We are working on creating a submodule.
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
https://bugzilla.redhat.com/show_bug.cgi?id=1478052
--- Comment #3 from mgans...@alice.de ---
Thanks for the review.
Spec URL:
https://martinkg.fedorapeople.org/Review/SPECS/nuvola-app-google-play-music.spec
SRPM URL:
https://martinkg.fedorapeople.org/Review/SRPMS/nuvola-app-google-play-music-
https://bugzilla.redhat.com/show_bug.cgi?id=1344276
--- Comment #28 from Ken Dreyer ---
Please accept my apologies for the delay and lack of communication! That is
bad.
I can continue the review or let someone else take it.
Looks like the LICENSE file should be listed as %license under %files
https://bugzilla.redhat.com/show_bug.cgi?id=1427634
--- Comment #22 from Athos Ribeiro ---
Hi Fabio,
Sorry for the long wait here!
I am working on this review ATM. Yes, if you are shipping a bundled version for
RHEL, it would be nice to provide de bundled package. Is there a reason for
bundli
https://bugzilla.redhat.com/show_bug.cgi?id=1344276
--- Comment #29 from Sachidananda Urs ---
(In reply to Ken Dreyer from comment #28)
> Please accept my apologies for the delay and lack of communication! That is
> bad.
>
> I can continue the review or let someone else take it.
>
> Looks lik
https://bugzilla.redhat.com/show_bug.cgi?id=1374024
Fedora End Of Life changed:
What|Removed |Added
Status|ASSIGNED|CLOSED
Resolution|---
https://bugzilla.redhat.com/show_bug.cgi?id=1421048
--- Comment #3 from Robert-André Mauchin ---
Sorry but if you do a snapshot, you must include the date with the git
revision, as I pointed out in my previous comment:
https://fedoraproject.org/wiki/Packaging:Versioning#Snapshots
>That field m
https://bugzilla.redhat.com/show_bug.cgi?id=1479073
--- Comment #1 from Wart ---
Updated spec and src rpm to clean up a couple of rpmlint warnings:
Updated SPEC: https://wart.fedorapeople.org/tcl-tclnagios.spec
Updated SRPM: https://wart.fedorapeople.org/tcl-tclnagios-1.3-2.fc27.src.rpm
Build
https://bugzilla.redhat.com/show_bug.cgi?id=1462443
--- Comment #15 from Christoph Junghans ---
(In reply to Christoph Junghans from comment #14)
> > Patch was send to hdf5-ow...@fedoraproject.org.
> https://src.fedoraproject.org/rpms/hdf5/pull-request/1
And merged.
--
You are receiving this
https://bugzilla.redhat.com/show_bug.cgi?id=1479073
--- Comment #2 from Wart ---
Cleaned up the remaining meaningful rpmlint warnings:
SPEC: https://wart.fedorapeople.org/tcl-tclnagios.spec
SRPM: https://wart.fedorapeople.org/tcl-tclnagios-1.3-3.fc27.src.rpm
Scratch build: https://koji.fedora
https://bugzilla.redhat.com/show_bug.cgi?id=1409400
Bug 1409400 depends on bug 1409661, which changed state.
Bug 1409661 Summary: Backport %_global_cxxflags into F24 and F25
https://bugzilla.redhat.com/show_bug.cgi?id=1409661
What|Removed |Added
https://bugzilla.redhat.com/show_bug.cgi?id=1427634
--- Comment #23 from Fabio Valentini ---
Well ... last time I checked, golang (expecially golang(XXX) dependencies on
other golang-XXX-devel packages) was a mess on RHEL/EPEL, and that's why I
turned on bundled dependencies for RHEL builds.
H
https://bugzilla.redhat.com/show_bug.cgi?id=1450212
--- Comment #3 from Paul Whalen ---
Updated spec and srpm
SPEC: https://pwhalen.fedorapeople.org/packages/pyowm/pyowm.spec
SRPM:
https://pwhalen.fedorapeople.org/packages/pyowm/pyowm-2.6.1-3.fc26.src.rpm
--
You are receiving this mail becau
https://bugzilla.redhat.com/show_bug.cgi?id=1464194
--- Comment #8 from Dan Williams ---
(In reply to Rutvij Karkhanis from comment #7)
> We are working on creating a submodule.
Great, then we can close this request and work this all through updates to the
ixpdimm-sw packaging.
--
You are re
https://bugzilla.redhat.com/show_bug.cgi?id=1427634
--- Comment #24 from Athos Ribeiro ---
Thanks for the hard work packaging this, it is not a simple package at all :)
(In reply to Fabio Valentini from comment #15)
> Additionally, I have included a systemd preset file to *not* enable the
> sy
https://bugzilla.redhat.com/show_bug.cgi?id=1427634
--- Comment #25 from Fabio Valentini ---
> Thanks for the hard work packaging this, it is not a simple package at all :)
Yeah, it's not so easy, but hugo isn't simple, either. At least you don't have
to patch build scripts. syncthing upstream
https://bugzilla.redhat.com/show_bug.cgi?id=1478210
Mat Booth changed:
What|Removed |Added
CC||mat.bo...@redhat.com
--- Comment #3 from
https://bugzilla.redhat.com/show_bug.cgi?id=1478210
--- Comment #4 from Stuart D Gathman ---
The conditional macro doesn't help, because it won't build on EL7 (because of
the missing jars). I am building on Fedora 25, hence the bundled bouncycastle
is from Fedora 25 - with all the security pat
https://bugzilla.redhat.com/show_bug.cgi?id=1478210
--- Comment #5 from Stuart D Gathman ---
Spec URL: https://gathman.org/linux/SPECS/openas2.spec
SRPM URL: https://gathman.org/linux/f25/src/openas2-2.3.0-6.fc25.src.rpm
Moved logs to /var/log/openas2. Move config files to /etc/openas2. (Bot
https://bugzilla.redhat.com/show_bug.cgi?id=1478210
--- Comment #6 from Stuart D Gathman ---
Need to make configs readable by openas2, but not by the world (keys and
passwords). I don't see any other packages doing that with configs in /etc.
Maybe I should move configs back to /var/lib/openas
https://bugzilla.redhat.com/show_bug.cgi?id=1427634
--- Comment #26 from Athos Ribeiro ---
> syncthing upstream was a bit "uncooperative"
> about fixing some things ...
oh :(
> > Have you thought about pinging systemd packagers about this?
>
> I have, a month ago. Without any reaction so far
https://bugzilla.redhat.com/show_bug.cgi?id=1479027
--- Comment #2 from Athos Ribeiro ---
Sure! I will send the pull requests for the necessary updates later this week,
would do it today, but it seems part of our infra is down
Thanks for taking this review as well!
--
You are receiving this
https://bugzilla.redhat.com/show_bug.cgi?id=1404043
--- Comment #35 from Honggang LI ---
(In reply to Kevin Fenzi from comment #33)
> Slightly longer term, we have put in a PR to make lorax not prune perl:
> https://github.com/rhinstaller/lorax/pull/228 once thats merged and in a
> rawhide lo
https://bugzilla.redhat.com/show_bug.cgi?id=1404043
--- Comment #36 from Honggang LI ---
Created attachment 1310973
--> https://bugzilla.redhat.com/attachment.cgi?id=1310973&action=edit
Use %{?systemd_requires} macro
--
You are receiving this mail because:
You are on the CC list for the bug
https://bugzilla.redhat.com/show_bug.cgi?id=1478210
--- Comment #7 from Stuart D Gathman ---
jabberd seems to be a good package to emulate.
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_
https://bugzilla.redhat.com/show_bug.cgi?id=1427634
--- Comment #27 from Jan ONDREJ ---
(In reply to Fabio Valentini from comment #17)
> - 3 x E: hardcoded-library-path in */usr/lib/systemd/user-preset
>
> I can't fix this, because systemd's RPM macros don't provide a
> %{_userpresetdir} analog
https://bugzilla.redhat.com/show_bug.cgi?id=1473543
Sebastian Kisela changed:
What|Removed |Added
Status|NEW |CLOSED
Resolution|---
https://bugzilla.redhat.com/show_bug.cgi?id=1476584
Robert-André Mauchin changed:
What|Removed |Added
Status|NEW |ASSIGNED
--- Comment #6 from R
https://bugzilla.redhat.com/show_bug.cgi?id=1477567
--- Comment #5 from Michael Schwendt ---
Validating the appdata file inside the spec file is a MUST, btw:
https://fedoraproject.org/wiki/Packaging:Guidelines#AppData_files
https://fedoraproject.org/wiki/Packaging:AppData#app-data-validate_usag
https://bugzilla.redhat.com/show_bug.cgi?id=1476458
--- Comment #9 from Michael Schwendt ---
> Requires: openssl
https://fedoraproject.org/wiki/Packaging:Guidelines#Explicit_Requires
> %files
> %{_libdir}/*
Just from the looks of it and the -devel package, this is likely wrong.
See:
https:/
67 matches
Mail list logo