Could not quite bring myself to just say no.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/2249#issuecomment-1327216695
You are receiving this because you are subscribed to this thread.
Message ID:
Merged #2249 into master.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/2249#event-7891355010
You are receiving this because you are subscribed to this thread.
Message ID:
___
Rpm-maint
> Yes it happens a lot, in one highly specific corner. Which is often best
> served by having that specific corner handle it because it has its own
> wrinkles that do not apply elsewhere, like version comparing the name here.
>
> In principle, Debian versioning isn't _that_ different from rpm.
@frozencemetery Can you change `Also-authored-by` to `Co-authored-by`? The
latter is the generally recognized "magic tag" by Git commit processing
automation.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/2249#issuecomment-1318604927
Yes it happens a lot, in one highly specific corner. Which is often best served
by having that specific corner handle it because it has its own wrinkles that
do not apply elsewhere, like version comparing the name here.
In principle, Debian versioning isn't *that* different from rpm. If 'sort
Thanks for the review; updated.
> It's not a particularly common use-case AFAICS.
Well, depends on how you define "common", I suppose - it gets called every time
a new kernel is installed, which is pretty common :)
> Still, not convinced this needs to be in rpm.
I'm not seeing a better place
@frozencemetery pushed 1 commit.
b04fc4d2775470b659801252bd6ca87ccf36f03c Add rpmsort utility for sorting RPM
versions
--
View it on GitHub:
https://github.com/rpm-software-management/rpm/pull/2249/files/b2e6455060f8d4566eed5188e26f4d58c8183427..b04fc4d2775470b659801252bd6ca87ccf36f03c
You
Yeah there *are* use-cases, such as figuring out the order of stuff in a
directory. Still, not convinced this needs to be in rpm.
But assuming for a moment we were to merge this, there are some issues to sort
(pun not intended) out:
- this belongs to tools/ rather than the top-level directory
> I can see how grub* needs such a thing, but I don't really see why this
> should be in rpm itself. It's not a particularly common use-case AFAICS.
I had to hand-roll something similar when figuring out which was the most
recent VM kernel package I had installed in my dom0.
--
Reply to this
I can see how grub* needs such a thing, but I don't really see why this should
be in rpm itself. It's not a particularly common use-case AFAICS.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/2249#issuecomment-1296722814
You are
(CI was broken with 190354cb19af6ecdfec6f7eaa454d6afcf381128, hence the revert
commit.)
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/2249#issuecomment-1294023025
You are receiving this because you are subscribed to this thread.
@frozencemetery pushed 2 commits.
2718ab9f29692e0ef04da7d278b25e727f04ed71 Add rpm-sort utility for sorting RPM
versions
b2e6455060f8d4566eed5188e26f4d58c8183427 Revert "Add a blurb to hello.spec
cautioning against its use"
--
View it on GitHub:
12 matches
Mail list logo