[Bug 2064137] Review Request: rust-format-bytes - Macro to format bytestrings

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=2064137

Mads Kiilerich <mads@xxxxxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |mads@xxxxxxxxxxxxx



--- Comment #1 from Mads Kiilerich <mads@xxxxxxxxxxxxx> ---
I verified that the spec and source is exactly as generated with "rust2rpm -as
format-bytes", except

@@ -10,7 +10,8 @@
 Summary:        Macro to format bytestrings

 # Upstream license specification: None
-License:        # FIXME
+# Upstream license file: MIT
+License:        MIT

 URL:            https://crates.io/crates/format-bytes
 Source:         %{crates_source}
@@ -65,6 +66,8 @@

 %if %{with check}
 %check
+# the tests were generated with a different rust compiler version
+export TRYBUILD=overwrite
 %cargo_test
 %endif

The LICENCE file is MIT, which is OK.

The package builds locally on f35, installs, and works for enabling
experimental build of Mercurial with Rust extensions.

rpmlint is clean, except "W: no-documentation".


-- 
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=2064137
_______________________________________________
package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx
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@xxxxxxxxxxxxxxxxxxxxxxx
Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux