[Bug 1835452] Review Request: mlxbf-bootctl - Bootloader control for Mellanox BlueField

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1835452

Honggang LI  changed:

   What|Removed |Added

  Flags|needinfo?(ho...@redhat.com) |needinfo?(ahleihel@redhat.c
   ||om)



--- Comment #14 from Honggang LI  ---
(In reply to Alaa Hleihel (NVIDIA Mellanox) from comment #13)
> Hi, 
> 
> What is the status of adding this package to Fedora?

https://bugzilla.redhat.com/show_bug.cgi?id=1846139#c3

It seems the firmware will be rejected. Is it still necessary to package up
this tool for Fedora?


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1944392] Review Request: rust-build-env - Extract information about the build process from the environment

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1944392



--- Comment #12 from Fedora Update System  ---
FEDORA-2021-1cb125936f has been pushed to the Fedora 32 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf upgrade --enablerepo=updates-testing
--advisory=FEDORA-2021-1cb125936f`
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2021-1cb125936f

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information
on how to test updates.


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1953496] Review Request: R-bslib - Custom Bootstrap Sass Themes for shiny and rmarkdown

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1953496



--- Comment #2 from Elliott Sales de Andrade  ---
Spec URL: https://qulogic.fedorapeople.org/reviews/R-bslib/R-bslib.spec
SRPM URL:
https://qulogic.fedorapeople.org/reviews/R-bslib/R-bslib-0.2.4-2.fc33.src.rpm

Add explicit license breakdown


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1015253] Review Request: svn2git - A tool for migrating SVN projects to Git

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1015253

Sergio Basto  changed:

   What|Removed |Added

 CC||ser...@serjux.com



--- Comment #18 from Sergio Basto  ---
Can't build it for F33 it says

Executing(%build): /bin/sh -e /var/tmp/rpm-tmp.HfRvax
+ umask 022
+ cd /builddir/build/BUILD
+ cd svn2git-2.3.2
+ gem build svn2git.gemspec
WARNING:  See https://guides.rubygems.org/specification-reference/ for help
ERROR:  While executing gem ... (Gem::InvalidSpecificationException)
["Rakefile", "VERSION.yml"] are not files

Any help to fix this error ? 

Thank you


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1896742] Review Request: python3-radexreader - Reader for the RADEX RD1212 Geiger counter

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1896742

Andy Mender  changed:

   What|Removed |Added

 Status|ASSIGNED|POST



--- Comment #18 from Andy Mender  ---
Thanks for the fixes and sorry for the delay. In order to get this package
approved, you need you request the repository as mentioned here:
https://fedoraproject.org/wiki/Join_the_package_collection_maintainers?rd=PackageMaintainers/Join#Add_Package_to_Source_Code_Management_.28SCM.29_system_and_Set_Owner

Since you've been sponsored already, I'm moving this to POST.


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1958681] Review Request: rust-xi-unicode - Unicode utilities useful for text editing, including a line breaking iterator

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1958681



--- Comment #3 from Davide Cavalca  ---
Thanks!

$ fedpkg request-repo rust-xi-unicode 1958681
https://pagure.io/releng/fedora-scm-requests/issue/33866


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1958602] Review Request: rust-k9 - Rust testing library

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1958602



--- Comment #3 from Davide Cavalca  ---
Thanks!

$ fedpkg request-repo rust-k9 1958602
https://pagure.io/releng/fedora-scm-requests/issue/33865


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1958608] Review Request: rust-webpki - Web PKI X.509 Certificate Verification

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1958608



--- Comment #4 from Fabio Valentini  ---
Thanks for all your reviews!

https://pagure.io/releng/fedora-scm-requests/issue/33863
https://pagure.io/releng/fedora-scm-requests/issue/33864


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1958681] Review Request: rust-xi-unicode - Unicode utilities useful for text editing, including a line breaking iterator

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1958681

Robert-André Mauchin   changed:

   What|Removed |Added

 Status|NEW |POST
 CC||zebo...@gmail.com
   Assignee|nob...@fedoraproject.org|zebo...@gmail.com
   Doc Type|--- |If docs needed, set a value
  Flags||fedora-review+



--- Comment #2 from Robert-André Mauchin   ---
 - Ask upstream to include a license in their crate


 - License ok
 - Latest version packaged
 - Builds in mock
 - No rpmlint errors
 - Conforms to Packaging Guidelines

Package approved.


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1958608] Review Request: rust-webpki - Web PKI X.509 Certificate Verification

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1958608

Robert-André Mauchin   changed:

   What|Removed |Added

 Status|NEW |POST
 CC||zebo...@gmail.com
   Assignee|nob...@fedoraproject.org|zebo...@gmail.com
  Flags||fedora-review+



--- Comment #3 from Robert-André Mauchin   ---
 - License ok
 - Latest version packaged
 - Builds in mock
 - No rpmlint errors
 - Conforms to Packaging Guidelines

Package approved.


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1958602] Review Request: rust-k9 - Rust testing library

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1958602

Robert-André Mauchin   changed:

   What|Removed |Added

 Status|NEW |POST
 CC||zebo...@gmail.com
   Assignee|nob...@fedoraproject.org|zebo...@gmail.com
   Doc Type|--- |If docs needed, set a value
  Flags||fedora-review+



--- Comment #2 from Robert-André Mauchin   ---
  - Ask upstream to include a LICENSE file in the crate


 - License ok
 - Latest version packaged
 - Builds in mock
 - No rpmlint errors
 - Conforms to Packaging Guidelines

Package approved.


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1958330] Review Request: rust-libbpf-sys - Rust bindings to libbpf from the Linux kernel

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1958330



--- Comment #5 from Robert-André Mauchin   ---
I,ve changed this to:

diff -Naur a/bindings.h b/bindings.h
--- a/bindings.h2021-01-19 11:12:47.0 -0800
+++ b/bindings.h2021-05-09 10:40:12.043397710 -0700
@@ -1,8 +1,8 @@
-#include "libbpf/include/uapi/linux/if_link.h"
-#include "libbpf/src/bpf.h"
-#include "libbpf/src/btf.h"
-#include "libbpf/src/libbpf.h"
-#include "libbpf/src/xsk.h"
+#include 
+#include 
+#include 
+#include 
+#include 

 extern __u64 *_xsk_ring_prod__fill_addr(struct xsk_ring_prod *fill, __u32
idx);

@@ -32,4 +32,4 @@

 extern __u64 _xsk_umem__extract_offset(__u64 addr);

-extern __u64 _xsk_umem__add_offset_to_addr(__u64 addr);
\ No newline at end of file
+extern __u64 _xsk_umem__add_offset_to_addr(__u64 addr);
diff -Naur a/build.rs b/build.rs
--- a/build.rs  2021-01-19 11:12:47.0 -0800
+++ b/build.rs  2021-05-09 10:41:00.968869985 -0700
@@ -10,39 +10,16 @@ fn main() {
 let out_dir_str = out_dir.to_str().unwrap();

 if cfg!(target_os = "linux") {
-let status = Command::new("make")
-.arg("install")
-.env("BUILD_STATIC_ONLY", "y")
-.env("PREFIX", "/")
-.env("LIBDIR", "")
-.env("OBJDIR", out_dir.join("obj").to_str().unwrap())
-.env("DESTDIR", out_dir_str)
-.env("CFLAGS", "-g -O2 -Werror -Wall -fPIC")
-.current_dir(src_dir.join("libbpf/src"))
-.status()
-.unwrap();
-
-assert!(status.success());
-
-let status = Command::new("make")
-.arg("clean")
-.current_dir(src_dir.join("libbpf/src"))
-.status()
-.unwrap();
-
-assert!(status.success());
-
 cc::Build::new()
 .file("bindings.c")
-.include(src_dir.join("libbpf/include"))
-.include(src_dir.join("libbpf/include/uapi"))
 .out_dir(out_dir_str)
 .compile("bindings");

 println!("cargo:rustc-link-search=native={}", out_dir_str);
 println!("cargo:rustc-link-lib=elf");
 println!("cargo:rustc-link-lib=z");
-println!("cargo:rustc-link-lib=static=bpf");
+println!("cargo:rustc-link-lib=static=bpf");
+println!("cargo:rustc-link-search=native={}", "/usr/lib64");
 println!("cargo:include={}/include", out_dir_str);
 }
 }


and it works but it shouldn't be hard-coded to "/usr/lib64", Maybe passed
through an environment variable?


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1913870] Review Request: qvge - visual graph editor

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1913870

Eugene A. Pivnev  changed:

   What|Removed |Added

  Flags|needinfo?(ti.eugene@gmail.c |
   |om) |



--- Comment #17 from Eugene A. Pivnev  ---
(In reply to Ben Beasley from comment #16 and Otto Urpelainen from comment #15)

Ok, let's go next level:
Spec: https://tieugene.fedorapeople.org/rpms/qvge/qvge.spec
SRPM:
https://tieugene.fedorapeople.org/rpms/qvge/qvge-0.6.2-4.20210405git0fd4648.fc34.src.rpm
Koji: https://koji.fedoraproject.org/koji/taskinfo?taskID=67573314

My comments:

> 1. I think the License should be "(MIT and LGPLv3 and BSD)"
After thoughtful reading of "Mixed Source Licensing Scenario" and its sample I
decided to use 'MIT and (BSD and LGPLv3)' as main sources are MIT-licensed.

> 2. all the licenses and copyright notices included in %license
Done

> 3. qvge.appdata.xml must also be updated to list the actual license that has 
> now been determined
Done

> 4. Add "BuildRequires: make"
Done

> 5. add "Requires: shared-mime-info"
Done

> 6. There is no man page, you should get in contact with the upstream about 
> adding such
I cannot imagine what to write into manpage.
"man qvge: 'to run qvge just run qvge'"?
It has no any CLI option.
> Agreed, although I will note that this is a SHOULD rather than a MUST, and 
> that help2man can be useful sometimes.
and -h not works too.

> 7. %check is recommended, perhaps add a comment line there explaining that 
> upstream does not provide any?
Noted in %check section. It not provides 'make check'


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1958681] New: Review Request: rust-xi-unicode - Unicode utilities useful for text editing, including a line breaking iterator

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1958681

Bug ID: 1958681
   Summary: Review Request: rust-xi-unicode - Unicode utilities
useful for text editing, including a line breaking
iterator
   Product: Fedora
   Version: rawhide
Status: NEW
 Component: Package Review
  Assignee: nob...@fedoraproject.org
  Reporter: dcava...@fb.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org
  Target Milestone: ---
Classification: Fedora



Spec URL:
https://dcavalca.fedorapeople.org/review/rust-xi-unicode/rust-xi-unicode.spec
SRPM URL:
https://dcavalca.fedorapeople.org/review/rust-xi-unicode/rust-xi-unicode-0.3.0-1.fc35.src.rpm

Description:
Unicode utilities useful for text editing, including a line breaking iterator.

Fedora Account System Username: dcavalca


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1958330] Review Request: rust-libbpf-sys - Rust bindings to libbpf from the Linux kernel

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1958330



--- Comment #4 from Robert-André Mauchin   ---
-println!("cargo:rustc-link-lib=static=bpf");
+println!("cargo:rustc-link-lib=bpf");

This still should stay static I believe and depend on libbpf-static


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1958564] Review Request: python-overpy - Python Wrapper to access the Overpass API

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1958564

Robert-André Mauchin   changed:

   What|Removed |Added

 Status|ASSIGNED|POST
  Flags|fedora-review?  |fedora-review+



--- Comment #6 from Robert-André Mauchin   ---
Package approved.


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1958366] Review Request: rust-ncurses - Very thin wrapper around the ncurses TUI library

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1958366



--- Comment #3 from Davide Cavalca  ---
Thanks!

$ fedpkg request-repo rust-ncurses 1958366
https://pagure.io/releng/fedora-scm-requests/issue/33862


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1958366] Review Request: rust-ncurses - Very thin wrapper around the ncurses TUI library

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1958366

Robert-André Mauchin   changed:

   What|Removed |Added

 Status|NEW |POST
 CC||zebo...@gmail.com
   Assignee|nob...@fedoraproject.org|zebo...@gmail.com
   Doc Type|--- |If docs needed, set a value
  Flags||fedora-review+



--- Comment #2 from Robert-André Mauchin   ---
 - You must install the LICENSE with %license in %files

%packagedevel
Summary:%{summary}
BuildArch:  noarch

%descriptiondevel %{_description}

This package contains library source intended for building other packages
which use "%{crate}" crate.

%files  devel
%license LICENSE
%doc README.md
%{cargo_registry}/%{crate}-%{version_no_tilde}/

 - License ok
 - Latest version packaged
 - Builds in mock
 - No rpmlint errors
 - Conforms to Packaging Guidelines

Package approved. Please fix the aforementioned issue before import.


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1958330] Review Request: rust-libbpf-sys - Rust bindings to libbpf from the Linux kernel

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1958330



--- Comment #3 from Davide Cavalca  ---
Spec URL:
https://dcavalca.fedorapeople.org/review/rust-libbpf-sys/rust-libbpf-sys.spec
SRPM URL:
https://dcavalca.fedorapeople.org/review/rust-libbpf-sys/rust-libbpf-sys-0.3.0~1-2.fc35.src.rpm

Changelog:
- Install license
- Unbundle libbpf and use the system one instead
- Add missing Requires to devel subpackages


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1958352] Review Request: rust-libbpf-rs - Safe, idiomatic, and opinionated wrapper around libbpf-sys

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1958352

Robert-André Mauchin   changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||zebo...@gmail.com
   Assignee|nob...@fedoraproject.org|zebo...@gmail.com
  Flags||fedora-review?



--- Comment #1 from Robert-André Mauchin   ---
 - Please add a comment explaining what changes you made in Cargo.toml

# Initial patched metadata
Patch0: libbpf-rs-fix-metadata.diff

 - This shouldn't be necessary if you made the changes I told you about in the
-sys package:

BuildRequires:  elfutils-libelf-devel
BuildRequires:  zlib-devel - Please ask upstream to add a LICENSE file in the
crate


 - Please ask upstream to add a license file into the crate


 - License ok
 - Latest version packaged
 - Builds in mock
 - No rpmlint errors
 - Conforms to Packaging Guidelines


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1954236] Review Request: golang-uber-automaxprocs - Automatically set GOMAXPROCS to match Linux container CPU quota

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1954236



--- Comment #3 from Olivier Lemasle  ---
Thanks for the review Robert-André

https://pagure.io/releng/fedora-scm-requests/issue/33861


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1958348] Review Request: rust-num_enum - Procedural macros to make inter-operation between primitives and enums easier

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1958348



--- Comment #2 from Davide Cavalca  ---
Thanks!

$ fedpkg request-repo rust-num_enum 1958348
https://pagure.io/releng/fedora-scm-requests/issue/33860


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1958338] Review Request: rust-num_enum_derive - Internal implementation details for ::num_enum (Procedural macros to make inter-operation between primitives and enums easier)

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1958338



--- Comment #3 from Davide Cavalca  ---
Thanks!

$ fedpkg request-repo rust-num_enum_derive 1958338 
https://pagure.io/releng/fedora-scm-requests/issue/33859


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1958333] Review Request: rust-vsprintf - Rust bindings to the libc vsprintf function

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1958333



--- Comment #3 from Davide Cavalca  ---
Thanks!

$ fedpkg request-repo rust-vsprintf 1958333 
https://pagure.io/releng/fedora-scm-requests/issue/33858


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1958307] Review Request: rust-blocking - Thread pool for isolating blocking I/O in async programs

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1958307



--- Comment #2 from Davide Cavalca  ---
Thanks!

$ fedpkg request-repo rust-blocking 1958307 
https://pagure.io/releng/fedora-scm-requests/issue/33857


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1958348] Review Request: rust-num_enum - Procedural macros to make inter-operation between primitives and enums easier

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1958348

Robert-André Mauchin   changed:

   What|Removed |Added

 Status|NEW |POST
 CC||zebo...@gmail.com
   Assignee|nob...@fedoraproject.org|zebo...@gmail.com
  Flags||fedora-review+



--- Comment #1 from Robert-André Mauchin   ---
 - You must install the LICENSE with %license in %files

%packagedevel
Summary:%{summary}
BuildArch:  noarch

%descriptiondevel %{_description}

This package contains library source intended for building other packages
which use "%{crate}" crate.

%files  devel
%license LICENSE
%doc README.md
%{cargo_registry}/%{crate}-%{version_no_tilde}/

 - License ok
 - Latest version packaged
 - Builds in mock
 - No rpmlint errors
 - Conforms to Packaging Guidelines

Package approved. Please fix the aforementioned issue before import.


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1957973] Review Request: rust-atomic-waker - Synchronization primitive for task wakeup

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1957973



--- Comment #3 from Davide Cavalca  ---
Thanks!

$ fedpkg request-repo rust-atomic-waker 1957973
https://pagure.io/releng/fedora-scm-requests/issue/33856


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1958338] Review Request: rust-num_enum_derive - Internal implementation details for ::num_enum (Procedural macros to make inter-operation between primitives and enums easier)

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1958338

Robert-André Mauchin   changed:

   What|Removed |Added

 Status|NEW |POST
 CC||zebo...@gmail.com
   Assignee|nob...@fedoraproject.org|zebo...@gmail.com
   Doc Type|--- |If docs needed, set a value
  Flags||fedora-review+



--- Comment #2 from Robert-André Mauchin   ---
 - Summary must be below 80 characters

Summary:Internal implementation details for ::num_enum

 - You must install the LICENSE with %license in %files

%packagedevel
Summary:%{summary}
BuildArch:  noarch

%descriptiondevel %{_description}

This package contains library source intended for building other packages
which use "%{crate}" crate.

%files  devel
%license LICENSE
%doc README.md
%{cargo_registry}/%{crate}-%{version_no_tilde}/

 - License ok
 - Latest version packaged
 - Builds in mock
 - No rpmlint errors
 - Conforms to Packaging Guidelines

Package approved. Please fix the aforementioned issues before import.


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1958333] Review Request: rust-vsprintf - Rust bindings to the libc vsprintf function

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1958333

Robert-André Mauchin   changed:

   What|Removed |Added

 Status|NEW |POST
 CC||zebo...@gmail.com
   Assignee|nob...@fedoraproject.org|zebo...@gmail.com
   Doc Type|--- |If docs needed, set a value
  Flags||fedora-review+



--- Comment #2 from Robert-André Mauchin   ---
 - You must install the LICENSE with %license in %files

%packagedevel
Summary:%{summary}
BuildArch:  noarch

%descriptiondevel %{_description}

This package contains library source intended for building other packages
which use "%{crate}" crate.

%files  devel
%license LICENSE
%doc README.md
%{cargo_registry}/%{crate}-%{version_no_tilde}/

 - License ok
 - Latest version packaged
 - Builds in mock
 - No rpmlint errors
 - Conforms to Packaging Guidelines

Package approved. Please fix the aforementioned issue before import.


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1958564] Review Request: python-overpy - Python Wrapper to access the Overpass API

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1958564



--- Comment #5 from Iztok Fister Jr.  ---
Thanks again. Fixed now.

SPEC:
https://raw.githubusercontent.com/firefly-cpp/rpm-overpy/main/python-overpy.spec


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1958330] Review Request: rust-libbpf-sys - Rust bindings to libbpf from the Linux kernel

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1958330

Robert-André Mauchin   changed:

   What|Removed |Added

 CC||zebo...@gmail.com
   Doc Type|--- |If docs needed, set a value



--- Comment #2 from Robert-André Mauchin   ---
 - You must install the LICENSE with %license in %files

%packagedevel
Summary:%{summary}
BuildArch:  noarch

%descriptiondevel %{_description}

This package contains library source intended for building other packages
which use "%{crate}" crate.

%files  devel
%license LICENSE
%doc README.md
%{cargo_registry}/%{crate}-%{version_no_tilde}/


 - Can't you unbundle libbpf? We do have a libbpf-static in Fedora.

 - These should be required in the devel subpackages too:

%packagedevel
Summary:%{summary}
BuildArch:  noarch
Requires:   elfutils-libelf-devel
Requires:   zlib-devel

so that when a package needs rust-libbpf-sys to build, it can fetch its lib
dependencies.


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1950923] Review Request: python-fireflyalgorithm - Implementation of Firefly Algorithm in Python

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1950923



--- Comment #8 from Iztok Fister Jr.  ---
Thanks to both of you.

It is now working as expected.

[iztok@localhost noarch]$ sudo dnf install
./python3-fireflyalgorithm-0.0.4-2.fc33.noarch.rpm
Last metadata expiration check: 4:55:14 ago on ned 09 maj 2021 13:12:01.
Dependencies resolved.
==
 Package   Architecture
   VersionRepository   
 Size
==
Installing:
 python3-fireflyalgorithm  noarch  
   0.0.4-2.fc33   @commandline 
 13 k
 replacing  python3-FireflyAlgorithm.noarch 0.0.4-1.fc33

Transaction Summary
==
Install  1 Package


New files are online:

SPEC:
https://raw.githubusercontent.com/firefly-cpp/rpm-firefly/main/python-fireflyalgorithm.spec
SRPM:
https://github.com/firefly-cpp/rpm-firefly/raw/main/python-fireflyalgorithm-0.0.4-2.fc33.src.rpm


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1958039] Review Request: rust-epoll - Safe epoll interface

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1958039

Robert-André Mauchin   changed:

   What|Removed |Added

 Status|NEW |POST
 CC||zebo...@gmail.com
   Assignee|nob...@fedoraproject.org|zebo...@gmail.com
  Flags||fedora-review+



--- Comment #1 from Robert-André Mauchin   ---
 - License ok
 - Latest version packaged
 - Builds in mock
 - No rpmlint errors
 - Conforms to Packaging Guidelines

Package approved.


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1958307] Review Request: rust-blocking - Thread pool for isolating blocking I/O in async programs

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1958307

Robert-André Mauchin   changed:

   What|Removed |Added

 Status|NEW |POST
 CC||zebo...@gmail.com
   Assignee|nob...@fedoraproject.org|zebo...@gmail.com
  Flags||fedora-review+



--- Comment #1 from Robert-André Mauchin   ---
 - You must install the LICENSE with %license in %files

%packagedevel
Summary:%{summary}
BuildArch:  noarch

%descriptiondevel %{_description}

This package contains library source intended for building other packages
which use "%{crate}" crate.

%files  devel
%license LICENSE-APACHE LICENSE-MIT
%doc README.md
%{cargo_registry}/%{crate}-%{version_no_tilde}/


 - License ok
 - Latest version packaged
 - Builds in mock
 - No rpmlint errors
 - Conforms to Packaging Guidelines

Package approved. Please fix the aforementioned issue before import.


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1957973] Review Request: rust-atomic-waker - Synchronization primitive for task wakeup

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1957973

Robert-André Mauchin   changed:

   What|Removed |Added

 Status|NEW |POST
 CC||zebo...@gmail.com
   Assignee|nob...@fedoraproject.org|zebo...@gmail.com
   Doc Type|--- |If docs needed, set a value
  Flags||fedora-review+



--- Comment #2 from Robert-André Mauchin   ---
 - You must install the licenses with %license in %files

%packagedevel
Summary:%{summary}
BuildArch:  noarch

%descriptiondevel %{_description}

This package contains library source intended for building other packages
which use "%{crate}" crate.

%files  devel
%license LICENSE-APACHE LICENSE-MIT LICENSE-THIRD-PARTY
%doc README.md
%{cargo_registry}/%{crate}-%{version_no_tilde}/


 - License ok
 - Latest version packaged
 - Builds in mock
 - No rpmlint errors
 - Conforms to Packaging Guidelines

Package approved. Please fix the aforementioned issue before import.


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1958304] Review Request: rust-async-executor - Async executor

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1958304



--- Comment #2 from Davide Cavalca  ---
Thanks!

$ fedpkg request-repo rust-async-executor 1958304
https://pagure.io/releng/fedora-scm-requests/issue/33854


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1957978] Review Request: rust-async-channel - Async multi-producer multi-consumer channel

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1957978



--- Comment #2 from Davide Cavalca  ---
Thanks!

$ fedpkg request-repo rust-async-channel 1957978 
https://pagure.io/releng/fedora-scm-requests/issue/33853


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1957963] Review Request: rust-event-listener - Notify async tasks or threads

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1957963



--- Comment #3 from Davide Cavalca  ---
Thanks!

$ fedpkg request-repo rust-event-listener 1957963 
https://pagure.io/releng/fedora-scm-requests/issue/33852


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1957911] Review Request: rust-ioctl-rs - Rust bindings for system ioctls

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1957911



--- Comment #3 from Davide Cavalca  ---
Thanks!

$ fedpkg request-repo rust-ioctl-rs 1957911 
https://pagure.io/releng/fedora-scm-requests/issue/33851


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1957907] Review Request: rust-serial-core - Rust abstractions for serial ports

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1957907



--- Comment #3 from Davide Cavalca  ---
Thanks!

$ fedpkg request-repo rust-serial-core 1957907 
https://pagure.io/releng/fedora-scm-requests/issue/33850


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1957903] Review Request: rust-filedescriptor - More ergonomic wrappers around RawFd and RawHandle

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1957903



--- Comment #3 from Davide Cavalca  ---
Thanks!

$ fedpkg request-repo rust-filedescriptor 1957903 
https://pagure.io/releng/fedora-scm-requests/issue/33849


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1958304] Review Request: rust-async-executor - Async executor

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1958304

Robert-André Mauchin   changed:

   What|Removed |Added

 Status|NEW |POST
 CC||zebo...@gmail.com
   Assignee|nob...@fedoraproject.org|zebo...@gmail.com
  Flags||fedora-review+



--- Comment #1 from Robert-André Mauchin   ---
 - You must install the licenses with %license in %files

%packagedevel
Summary:%{summary}
BuildArch:  noarch

%descriptiondevel %{_description}

This package contains library source intended for building other packages
which use "%{crate}" crate.

%files  devel
%license LICENSE-APACHE LICENSE-MIT
%doc README.md
%{cargo_registry}/%{crate}-%{version_no_tilde}/


 - License ok
 - Latest version packaged
 - Builds in mock
 - No rpmlint errors
 - Conforms to Packaging Guidelines

Package approved. Please fix the aforementioned issue before import.


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1957978] Review Request: rust-async-channel - Async multi-producer multi-consumer channel

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1957978

Robert-André Mauchin   changed:

   What|Removed |Added

 Status|NEW |POST
 CC||zebo...@gmail.com
   Assignee|nob...@fedoraproject.org|zebo...@gmail.com
  Flags||fedora-review+



--- Comment #1 from Robert-André Mauchin   ---
 - You must install the LICENSE with %license in %files

%packagedevel
Summary:%{summary}
BuildArch:  noarch

%descriptiondevel %{_description}

This package contains library source intended for building other packages
which use "%{crate}" crate.

%files  devel
%license LICENSE-APACHE LICENSE-MIT
%doc README.md
%{cargo_registry}/%{crate}-%{version_no_tilde}/


 - License ok
 - Latest version packaged
 - Builds in mock
 - No rpmlint errors
 - Conforms to Packaging Guidelines

Package approved. Please fix the aforementioned issue before import.


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1957963] Review Request: rust-event-listener - Notify async tasks or threads

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1957963

Robert-André Mauchin   changed:

   What|Removed |Added

 Status|NEW |POST
 CC||zebo...@gmail.com
   Assignee|nob...@fedoraproject.org|zebo...@gmail.com
   Doc Type|--- |If docs needed, set a value
  Flags||fedora-review+



--- Comment #2 from Robert-André Mauchin   ---
 - You must install the LICENSE with %license in %files

%packagedevel
Summary:%{summary}
BuildArch:  noarch

%descriptiondevel %{_description}

This package contains library source intended for building other packages
which use "%{crate}" crate.

%files  devel
%license LICENSE-APACHE LICENSE-MIT
%doc README.md
%{cargo_registry}/%{crate}-%{version_no_tilde}/


 - License ok
 - Latest version packaged
 - Builds in mock
 - No rpmlint errors
 - Conforms to Packaging Guidelines

Package approved. Please fix the aforementioned issue before import.


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1957911] Review Request: rust-ioctl-rs - Rust bindings for system ioctls

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1957911

Robert-André Mauchin   changed:

   What|Removed |Added

 Status|NEW |POST
 CC||zebo...@gmail.com
   Assignee|nob...@fedoraproject.org|zebo...@gmail.com
   Doc Type|--- |If docs needed, set a value
  Flags||fedora-review+



--- Comment #2 from Robert-André Mauchin   ---


 - You must install the LICENSE with %license in %files

%packagedevel
Summary:%{summary}
BuildArch:  noarch

%descriptiondevel %{_description}

This package contains library source intended for building other packages
which use "%{crate}" crate.

%files  devel
%license LICENSE
%doc README.md
%{cargo_registry}/%{crate}-%{version_no_tilde}/


 - License ok
 - Latest version packaged
 - Builds in mock
 - No rpmlint errors
 - Conforms to Packaging Guidelines

Package approved. Please fix the aforementioned issue before import.


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1957907] Review Request: rust-serial-core - Rust abstractions for serial ports

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1957907

Robert-André Mauchin   changed:

   What|Removed |Added

 Status|NEW |POST
 CC||zebo...@gmail.com
   Assignee|nob...@fedoraproject.org|zebo...@gmail.com
   Doc Type|--- |If docs needed, set a value
  Flags||fedora-review+



--- Comment #2 from Robert-André Mauchin   ---
 - You must install the LICENSE with %license in %files

%packagedevel
Summary:%{summary}
BuildArch:  noarch

%descriptiondevel %{_description}

This package contains library source intended for building other packages
which use "%{crate}" crate.

%files  devel
%license LICENSE
%doc README.md
%{cargo_registry}/%{crate}-%{version_no_tilde}/

 - License ok
 - Latest version packaged
 - Builds in mock
 - No rpmlint errors
 - Conforms to Packaging Guidelines

Package approved. Please fix the aforementioned issue before import.


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1957903] Review Request: rust-filedescriptor - More ergonomic wrappers around RawFd and RawHandle

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1957903

Robert-André Mauchin   changed:

   What|Removed |Added

 Status|NEW |POST
 CC||zebo...@gmail.com
   Assignee|nob...@fedoraproject.org|zebo...@gmail.com
   Doc Type|--- |If docs needed, set a value
  Flags||fedora-review+



--- Comment #2 from Robert-André Mauchin   ---
 - Ask upstream to include a license file in the crate


 - License ok
 - Latest version packaged
 - Builds in mock
 - No rpmlint errors
 - Conforms to Packaging Guidelines

Package approved.


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1956240] Review Request: rust-edit - Open a file in the default text editor regardless of platform

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1956240

Robert-André Mauchin   changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||zebo...@gmail.com
   Assignee|nob...@fedoraproject.org|zebo...@gmail.com
   Doc Type|--- |If docs needed, set a value
  Flags||fedora-review?



--- Comment #1 from Robert-André Mauchin   ---
 - You must install the LICENSE file with %license in %files


%if ! %{__cargo_skip_build}
%package -n %{crate}
Summary:%{summary}

%description -n %{crate} %{_description}

%files   -n %{crate}
%license LICENSE
%doc README.md
%endif

%packagedevel
Summary:%{summary}
BuildArch:  noarch

%descriptiondevel %{_description}

This package contains library source intended for building other packages
which use "%{crate}" crate.

%files  devel
%license LICENSE
%doc README.md
%{cargo_registry}/%{crate}-%{version_no_tilde}/

 - Please note that you can edit the Cargo.toml with "rust2rpm -p"

 - License ok
 - Latest version packaged
 - Builds in mock
 - No rpmlint errors
 - Conforms to Packaging Guidelines


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1956558] Review Request: rust-derive_builder_macro - Rust macro to automatically implement the builder pattern for arbitrary structs

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1956558



--- Comment #3 from Fabio Valentini  ---
Thanks for the review!

https://pagure.io/releng/fedora-scm-requests/issue/33847
https://pagure.io/releng/fedora-scm-requests/issue/33848


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1956558] Review Request: rust-derive_builder_macro - Rust macro to automatically implement the builder pattern for arbitrary structs

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1956558

Robert-André Mauchin   changed:

   What|Removed |Added

 Status|NEW |POST
 CC||zebo...@gmail.com
   Assignee|nob...@fedoraproject.org|zebo...@gmail.com
  Flags||fedora-review+



--- Comment #2 from Robert-André Mauchin   ---
 - License ok
 - Latest version packaged
 - Builds in mock
 - No rpmlint errors
 - Conforms to Packaging Guidelines

Package approved.


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1955944] Review Request: rust-value-bag - Anonymous structured values

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1955944



--- Comment #3 from Fabio Valentini  ---
Thanks for the review!

https://pagure.io/releng/fedora-scm-requests/issue/33845
https://pagure.io/releng/fedora-scm-requests/issue/33846


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1955944] Review Request: rust-value-bag - Anonymous structured values

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1955944

Robert-André Mauchin   changed:

   What|Removed |Added

 Status|NEW |POST
 CC||zebo...@gmail.com
   Assignee|nob...@fedoraproject.org|zebo...@gmail.com
  Flags||fedora-review+



--- Comment #2 from Robert-André Mauchin   ---
 - License ok
 - Latest version packaged
 - Builds in mock
 - No rpmlint errors
 - Conforms to Packaging Guidelines

Package approved.


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1950923] Review Request: python-fireflyalgorithm - Implementation of Firefly Algorithm in Python

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1950923

Robert-André Mauchin   changed:

   What|Removed |Added

 CC||zebo...@gmail.com



--- Comment #7 from Robert-André Mauchin   ---
(Per Iztok request)


Provides: python-FireflyAlgorithm = %{version}-%{release}
Obsoletes: python-FireflyAlgorithm < 0.0.4-2

is good, but you should add one for the Python 3 subpackage as this is what is
gonna be installed by the end user:

%package -n python3-%{new_name}
Summary:%{summary}
BuildRequires:  python3-devel
BuildRequires:  python3-setuptools

Provides: python3-FireflyAlgorithm = %{version}-%{release}
Obsoletes: python3-FireflyAlgorithm < 0.0.4-2

%description -n python3-%{new_name} %_description


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1958564] Review Request: python-overpy - Python Wrapper to access the Overpass API

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1958564



--- Comment #4 from Robert-André Mauchin   ---
This is not fixed:

 - The URL is still missing the repo name:

URL:https://github.com/DinoTools/python-overpy


 - You've built the docs but you haven't installed them in %doc


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1955946] Review Request: rust-wasm-bindgen-test - Internal testing crate for wasm-bindgen

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1955946



--- Comment #3 from Fabio Valentini  ---
Thanks for the review!

https://pagure.io/releng/fedora-scm-requests/issue/33843
https://pagure.io/releng/fedora-scm-requests/issue/33844


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1955946] Review Request: rust-wasm-bindgen-test - Internal testing crate for wasm-bindgen

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1955946

Robert-André Mauchin   changed:

   What|Removed |Added

 Status|NEW |POST
 CC||zebo...@gmail.com
   Assignee|nob...@fedoraproject.org|zebo...@gmail.com
  Flags||fedora-review+



--- Comment #2 from Robert-André Mauchin   ---
 - License ok
 - Latest version packaged
 - Builds in mock
 - No rpmlint errors
 - Conforms to Packaging Guidelines

Package approved.


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1955947] Review Request: rust-wasm-bindgen-futures - Bridging the gap between Rust Futures and JavaScript Promises

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1955947



--- Comment #3 from Fabio Valentini  ---
Thanks for the review!

https://pagure.io/releng/fedora-scm-requests/issue/33841
https://pagure.io/releng/fedora-scm-requests/issue/33842


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1955948] Review Request: rust-console_error_panic_hook - Panic hook for WASM that logs panics to console.error

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1955948



--- Comment #3 from Fabio Valentini  ---
Thanks for the review!

https://pagure.io/releng/fedora-scm-requests/issue/33839
https://pagure.io/releng/fedora-scm-requests/issue/33840


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1955943] Review Request: rust-sval_json - JSON support for the sval serialization framework

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1955943



--- Comment #3 from Fabio Valentini  ---
Thanks for the review!

https://pagure.io/releng/fedora-scm-requests/issue/33837
https://pagure.io/releng/fedora-scm-requests/issue/33838


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1955948] Review Request: rust-console_error_panic_hook - Panic hook for WASM that logs panics to console.error

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1955948

Robert-André Mauchin   changed:

   What|Removed |Added

 Status|NEW |POST
 CC||zebo...@gmail.com
   Assignee|nob...@fedoraproject.org|zebo...@gmail.com
   Doc Type|--- |If docs needed, set a value
  Flags||fedora-review+



--- Comment #2 from Robert-André Mauchin   ---
 - License ok
 - Latest version packaged
 - Builds in mock
 - No rpmlint errors
 - Conforms to Packaging Guidelines

Package approved.


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1955947] Review Request: rust-wasm-bindgen-futures - Bridging the gap between Rust Futures and JavaScript Promises

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1955947

Robert-André Mauchin   changed:

   What|Removed |Added

 Status|NEW |POST
 CC||zebo...@gmail.com
   Assignee|nob...@fedoraproject.org|zebo...@gmail.com
   Doc Type|--- |If docs needed, set a value
  Flags||fedora-review+



--- Comment #2 from Robert-André Mauchin   ---
 - License ok
 - Latest version packaged
 - Builds in mock
 - No rpmlint errors
 - Conforms to Packaging Guidelines

Package approved.


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1955943] Review Request: rust-sval_json - JSON support for the sval serialization framework

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1955943

Robert-André Mauchin   changed:

   What|Removed |Added

 Status|NEW |POST
 CC||zebo...@gmail.com
   Assignee|nob...@fedoraproject.org|zebo...@gmail.com
  Flags||fedora-review+



--- Comment #2 from Robert-André Mauchin   ---
 - License ok
 - Latest version packaged
 - Builds in mock
 - No rpmlint errors
 - Conforms to Packaging Guidelines

Package approved.


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1954236] Review Request: golang-uber-automaxprocs - Automatically set GOMAXPROCS to match Linux container CPU quota

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1954236

Robert-André Mauchin   changed:

   What|Removed |Added

 Status|NEW |POST
 CC||zebo...@gmail.com
   Assignee|nob...@fedoraproject.org|zebo...@gmail.com
   Doc Type|--- |If docs needed, set a value
  Flags||fedora-review+



--- Comment #2 from Robert-André Mauchin   ---
 - License ok
 - Latest version packaged
 - Builds in mock
 - No rpmlint errors
 - Conforms to Packaging Guidelines

Package approved.


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1958564] Review Request: python-overpy - Python Wrapper to access the Overpass API

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1958564



--- Comment #3 from Iztok Fister Jr.  ---
Thank you very much for another review. Revision is Online:
https://raw.githubusercontent.com/firefly-cpp/rpm-overpy/main/python-overpy.spec


Koji scratch: https://koji.fedoraproject.org/koji/taskinfo?taskID=67552608


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1953254] Review Request: SLiMgui - an evolutionary simulation framework

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1953254



--- Comment #8 from Robert-André Mauchin   ---
Also explicitly add gcc-c++ as a BuildRequires


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1953254] Review Request: SLiMgui - an evolutionary simulation framework

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1953254

Robert-André Mauchin   changed:

   What|Removed |Added

 CC||zebo...@gmail.com



--- Comment #7 from Robert-André Mauchin   ---
 - Capitalize the Summary:

Summary:An evolutionary simulation framework

 - Use a better name for your archive:

Source0:   
https://github.com/MesserLab/SLiM/archive/v%{version}/%{name}-%{version}.tar.gz

 - The patch in the SRC.RPM seems broken, you could use a direct link for that:

# Fixes a compilation issue on Fedora 34; fixed upstream for next release.
#
https://github.com/MesserLab/SLiM/commit/7964f7911e1665a1ba0783d9136b3d3bff06d931
Patch0:
https://github.com/MesserLab/SLiM/commit/7964f7911e1665a1ba0783d9136b3d3bff06d931.patch

 - Don't mix Suse stuff in a Fedora SPEC

 - Constrain the version here:

BuildRequires:  qt5-qtbase-devel  >= 5.9.5

 - Are you sure the ExclusiveArch is needed, have you tested other arches?

 - I don't see why this is necessary:

# Fedora: these package versions are those that COPR is building against, and
thus if
# they change because of point releases in Qt5, the RPMs need to be rebuilt and
deployed.
# Qt is weird and doesn't allow older software to be used if a newer point
release is
# installed on the system.
%if 0%{?fedora}
%if 0%{?fedora} == 31
Requires: qt5-qtbase == 5.13.2
%else
%if 0%{?fedora} == 32
Requires: qt5-qtbase == 5.14.2
%else
# Qt5 5.15.2 is the final version; Qt6 is not tested with SLiM.
%if 0%{?fedora} >= 33
Requires: qt5-qtbase == 5.15.2
%endif
%endif
%endif
%else
# If not installing on Fedora
# Conditonal requires for RHEL (and CentOS)
%if 0%{?rhel} == 8
Requires: qt5-qtbase == 5.12.5
%else
# Conditional requires for openSUSE and SLE.
# openSUSE Leap 15.2 and SLE 15 (and all service packs)
%if %{defined suse_version}
%if %{defined suse_version} == 1500
Requires: libqt5-qtbase == 5.12.7
%else
# openSUSE Tumbleweed
%if %{defined suse_version} >= 1550
Requires: libqt5-qtbase == 5.15.2
%endif
%endif
%else
# The minimum version of Qt5 required to run SLiMgui is 5.9.5, however this
does not actually assure that the system will install the same version of Qt5
that it was built against; an end-user may install a more recent Qt5 if, for
isntance, openSUSE Leap 15.3 builds against Qt 5.12.7 and the user upgrades to
5.15.2 when it is available. This is strange, but we can't protect against
every edge-case.
Requires: qt5-qtbase >= 5.9.5
%endif
%endif
%endif

Point releases don't matter, what matters is the major soname version of the
libraries it was built against (5 for Qt 5). There is no need to rebuild
anything unless that soname is changed.

 - No:

%prep
tar -xf ../SOURCES/v%{version}.tar.gz # Decompresses to 'SLiM-[version macro]'

Either use %autosetup -p1 or %setup -q:

%prep
%autosetup -p1 -n SLiM-%{version}

Then remove all the ./SLiM-%{version}/ prefix as it is now the base directory
for the next instructions.

 - This isn't how you apply a patch:


# The patch is successfully applied, as demonstrated by successful builds on
Fedora 34 on Copr.
# rpmlint likes to complain because we're not using the patch macro.
patch ./SLiM-%{version}/eidos/robin_hood.h < ../SOURCES/fedora34.patch

If you use %autosetup -p1, the patching is automatically handled.
If you use %setup -q, then add:

%patch0 -p1

 - Use the %cmake macro to use Fedora default build flags:

%cmake -DBUILD_SLIMGUI=ON

 - Then use:

%cmake_build

instead of:

%make_build

 - Slight change due to the use of %cmake:

# Binaries
pushd %{_vpath_builddir}
install -p --mode=0755 --target-directory %{buildroot}%{_bindir} eidos SLiMgui
install -p --mode=0755 --no-target-directory slim
%{buildroot}%{_bindir}/SLiMcli # Rename the binary so that it does not conflict
with Simple Login Manager.
popd

 - Use install with -p to keep the files timestamps

 - Use macros instead of hard-coded directories:

/usr/bin/ -> %{_bindir}/
/usr/share/ -> %{_datadir}/

 - This is not needed:

%post
update-mime-database -n /usr/share/mime/
xdg-mime install --mode system
/usr/share/mime/packages/org.messerlab.slimgui-mime.xml



Proposed SPEC:


Name:   SLiMgui
Version:3.6
Release:5%{?dist}
Summary:An evolutionary simulation framework

License:GPLv3+
URL:https://messerlab.org/slim/
Source0:   
https://github.com/MesserLab/SLiM/archive/v%{version}/%{name}-%{version}.tar.gz

# Fixes a compilation issue on Fedora 34; fixed upstream for next release.
#
https://github.com/MesserLab/SLiM/commit/7964f7911e1665a1ba0783d9136b3d3bff06d931
Patch0:
https://github.com/MesserLab/SLiM/commit/7964f7911e1665a1ba0783d9136b3d3bff06d931.patch

BuildRequires:  cmake
BuildRequires:  desktop-file-utils
BuildRequires:  qt5-qtbase-devel >= 5.9.5
BuildRequires:  libappstream-glib
Requires:   hicolor-icon-theme
# This is 

[Bug 1958608] Review Request: rust-webpki - Web PKI X.509 Certificate Verification

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1958608

Fabio Valentini  changed:

   What|Removed |Added

   Doc Type|--- |If docs needed, set a value



--- Comment #2 from Fabio Valentini  ---
Note that I have not packaged the latest version 0.22.0 yet, since Rustls
(which is what I am packaging this for) still requires 0.21.x.


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1958608] Review Request: rust-webpki - Web PKI X.509 Certificate Verification

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1958608



--- Comment #1 from Fabio Valentini  ---
This package built on koji: 
https://koji.fedoraproject.org/koji/taskinfo?taskID=67541864


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1958608] New: Review Request: rust-webpki - Web PKI X.509 Certificate Verification

2021-05-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1958608

Bug ID: 1958608
   Summary: Review Request: rust-webpki - Web PKI X.509
Certificate Verification
   Product: Fedora
   Version: rawhide
Status: NEW
 Component: Package Review
  Assignee: nob...@fedoraproject.org
  Reporter: decatho...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org
  Target Milestone: ---
Classification: Fedora



Spec URL: https://decathorpe.fedorapeople.org/rust-webpki.spec
SRPM URL: https://decathorpe.fedorapeople.org/rust-webpki-0.21.4-1.fc34.src.rpm

Description:
Web PKI X.509 Certificate Verification.

Fedora Account System Username: decathorpe


-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure