Skip to content

Commit

Permalink
usbutils: update portable.patch (usbutils 017)
Browse files Browse the repository at this point in the history
Signed-off-by: Rui Chen <[email protected]>
  • Loading branch information
chenrui333 committed Oct 30, 2023
1 parent 0fe617f commit 12f3d34
Showing 1 changed file with 144 additions and 242 deletions.
Loading

3 comments on commit 12f3d34

@tormodvolden
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wonder why the authorship, X-Origin, and patch structure was removed here?

@Bo98
Copy link
Member

@Bo98 Bo98 commented on 12f3d34 Sep 2, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Presumably as it was cherry-pick applied and git diff was used instead of git format-patch, whereas before it was basically copy-pasted from https://patch-diff.githubusercontent.com/raw/gregkh/usbutils/pull/134.patch.

Tbh this really needs upstreaming going forward. If the patch breaks again it may be better to just drop support for it.

@tormodvolden
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Keeping the X-Origin could actually help getting it upstream, since people using it can figure out where it comes from, and maybe contribute. And yes, it would be good to get it upstream. However it would require meson expertise because upstream is switching from autotools to meson.

Please sign in to comment.