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

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

Bug ID: 2064137
           Summary: Review Request: rust-format-bytes - Macro to format
                    bytestrings
           Product: Fedora
           Version: rawhide
          Hardware: All
                OS: Linux
            Status: NEW
         Component: Package Review
          Severity: medium
          Priority: medium
          Assignee: [e-mail]
          Reporter: [e-mail]
        QA Contact: [e-mail]
                CC: [e-mail]
  Target Milestone: ---
    Classification: Fedora

Spec URL:
https://alebastr.fedorapeople.org/review/mercurial-rs/rust-format-bytes.spec
SRPM URL:
https://alebastr.fedorapeople.org/review/mercurial-rs/rust-format-bytes-0.3....
Description:
Macro to format bytestrings.
Fedora Account System Username: alebastr

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

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

Aleksei Bavshin  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
         Depends On|                            |2064136
           Doc Type|---                         |If docs needed, set a value

Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=2064136
[Bug 2064136] Review Request: rust-format-bytes-macros - Macros for the
format-bytes crate

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

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

Aleksei Bavshin  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Blocks|                            |2063941

Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=2063941
[Bug 2063941] Feature request: Enable Mercurial rust extensions

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

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

Mads Kiilerich  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #1 from Mads Kiilerich  ---
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".

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

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

Mads Kiilerich  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
              Flags|                            |fedora-review+

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

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

Mads Kiilerich  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Assignee|[e-mail]    |[e-mail]

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

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

--- Comment #2 from Gwyn Ciesla  ---
(fedscm-admin):  The Pagure repository was created at
https://src.fedoraproject.org/rpms/rust-format-bytes

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

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

Fedora Update System  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |MODIFIED

--- Comment #3 from Fedora Update System  ---
FEDORA-2022-c7ededf903 has been submitted as an update to Fedora 36.
https://bodhi.fedoraproject.org/updates/FEDORA-2022-c7ededf903

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

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

Fedora Update System  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|MODIFIED                    |ON_QA

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

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

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

https://bugzilla.redhat.com/show_bug.cgi?id=2064137
Bug 2064137 depends on bug 2064136, which changed state.

Bug 2064136 Summary: Review Request: rust-format-bytes-macros - Macros for the format-bytes crate
https://bugzilla.redhat.com/show_bug.cgi?id=2064136

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ON_QA                       |CLOSED
         Resolution|---                         |ERRATA

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

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

Fedora Update System  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |ERRATA
             Status|ON_QA                       |CLOSED
        Last Closed|                            |2022-05-07 04:17:35

--- Comment #5 from Fedora Update System  ---
FEDORA-2022-c7ededf903 has been pushed to the Fedora 36 stable repository.
If problem still persists, please make note of it in this bug report.