Re: [Rpm-maint] [rpm-software-management/rpm] fix:possible null pointer reference (PR #2544)

2023-06-13 Thread Panu Matilainen
Closed #2544. -- Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/pull/2544#event-9524295387 You are receiving this because you are subscribed to this thread. Message ID: ___ Rpm-maint mailing list Rpm-

Re: [Rpm-maint] [rpm-software-management/rpm] fix:possible null pointer reference (PR #2544)

2023-06-13 Thread Panu Matilainen
Thanks for the patch! I applied manully (8ec9cfc81c3f8326a1c7ee5b795ef517cd48e6b4) to tweak the commit message a little - it's important to mention the affected subsystem in the summary so people can immediately tell whether it's relevant to them or not. -- Reply to this email directly or view

[Rpm-maint] [rpm-software-management/rpm] fix:possible null pointer reference (PR #2544)

2023-06-13 Thread slark-yuxj
slot1 and slot2 may be NULL You can view, comment on, or merge this pull request online at: https://github.com/rpm-software-management/rpm/pull/2544 -- Commit Summary -- * fix:possible null pointer reference -- File Changes -- M lib/backend/ndb/rpmxdb.c (2) -- Patch Links -- https://

Re: [Rpm-maint] [rpm-software-management/rpm] RPM v6 package format, first public draft for commenting (Discussion #2374)

2023-06-13 Thread Daniel Alley
@Conan-Kudo Is this making any sense? -- Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/discussions/2374#discussioncomment-6167128 You are receiving this because you are subscribed to this thread. Message ID: ___

[Rpm-maint] [rpm-software-management/rpm] A coredump may occur when the RPM package is installed (Issue #2543)

2023-06-13 Thread xujing
When I try to add a plugin that uses strcmp, a core occurs. The stack is as follows: ``` #0 ... #1 rpmpluginsCallFsmFilePost() #2 fsmDoMkdir() #3 ensureDir() #4 rpmPackageFilesInstall() #5 rpmpsmUnpack() ... #13 rpmInstall() #14 main() ``` I tried to analyze it and found that the second parameter

Re: [Rpm-maint] [rpm-software-management/rpm] Add doc notes related to option processing (PR #2542)

2023-06-13 Thread Panu Matilainen
Merged #2542 into master. -- Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/pull/2542#event-9514648994 You are receiving this because you are subscribed to this thread. Message ID: ___ Rpm-maint maili

Re: [Rpm-maint] [rpm-software-management/rpm] Add doc notes related to option processing (PR #2542)

2023-06-13 Thread Panu Matilainen
@pmatilai commented on this pull request. > @@ -43,6 +43,15 @@ macros are available: | %{-f*} | if present at invocation, the argument to the last occurence of flag f | %1, %2, ...| the arguments themselves (after getopt(3) processing) +With rpm >= 4.17 and disabled option processing th

Re: [Rpm-maint] [rpm-software-management/rpm] Add doc notes related to option processing (PR #2542)

2023-06-13 Thread Panu Matilainen
@pmatilai commented on this pull request. > @@ -43,6 +43,15 @@ macros are available: | %{-f*} | if present at invocation, the argument to the last occurence of flag f | %1, %2, ...| the arguments themselves (after getopt(3) processing) +With rpm >= 4.17 and disabled option processing th

Re: [Rpm-maint] [rpm-software-management/rpm] Add doc notes related to option processing (PR #2542)

2023-06-13 Thread Ralf Habacker
@rhabacker commented on this pull request. > @@ -43,6 +43,15 @@ macros are available: | %{-f*} | if present at invocation, the argument to the last occurence of flag f | %1, %2, ...| the arguments themselves (after getopt(3) processing) +With rpm >= 4.17 and disabled option processing t

Re: [Rpm-maint] [rpm-software-management/rpm] Rename SPECPARTS to .SPECPARTS to make it less disruptive (PR #2533)

2023-06-13 Thread Panu Matilainen
Closed #2533. -- Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/pull/2533#event-9513257463 You are receiving this because you are subscribed to this thread. Message ID: ___ Rpm-maint mailing list Rpm-

Re: [Rpm-maint] [rpm-software-management/rpm] Rename SPECPARTS to .SPECPARTS to make it less disruptive (PR #2533)

2023-06-13 Thread Panu Matilainen
Replaced by #2534, thanks for the patch though! -- Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/pull/2533#issuecomment-1588867604 You are receiving this because you are subscribed to this thread. Message ID: ___

Re: [Rpm-maint] [rpm-software-management/rpm] SPECPARTS dir in %_builddir/%buildsubdir is leaking to setuptools package discovery (Issue #2532)

2023-06-13 Thread Panu Matilainen
Closed #2532 as completed via df8d1fb1b32becf61bbe0366bd7a1966157bf08f. -- Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/issues/2532#event-9513245937 You are receiving this because you are subscribed to this thread. Message ID: ___

Re: [Rpm-maint] [rpm-software-management/rpm] Add %specpartsdir to macros.in (PR #2534)

2023-06-13 Thread Panu Matilainen
Merged #2534 into master. -- Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/pull/2534#event-9513245357 You are receiving this because you are subscribed to this thread. Message ID: ___ Rpm-maint maili

Re: [Rpm-maint] [rpm-software-management/rpm] Add doc notes related to option processing (PR #2542)

2023-06-13 Thread Panu Matilainen
@pmatilai commented on this pull request. > @@ -43,6 +43,15 @@ macros are available: | %{-f*} | if present at invocation, the argument to the last occurence of flag f | %1, %2, ...| the arguments themselves (after getopt(3) processing) +With rpm >= 4.17 and disabled option processing th

Re: [Rpm-maint] [rpm-software-management/rpm] Add %specpartsdir to macros.in (PR #2534)

2023-06-13 Thread Florian Festi
Adjusted the docs to the new location and added Resolved tag to the original ticket in the commit message. -- Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/pull/2534#issuecomment-1588856898 You are receiving this because you are subscribed to t

Re: [Rpm-maint] [rpm-software-management/rpm] Add %specpartsdir to macros.in (PR #2534)

2023-06-13 Thread Florian Festi
@ffesti pushed 1 commit. 252fc02ea530aaa9437f56a7d2e259ee0fbb2298 Add %specpartsdir to macros.in -- View it on GitHub: https://github.com/rpm-software-management/rpm/pull/2534/files/0e923736841f30d20c29474c79461759b66ac519..252fc02ea530aaa9437f56a7d2e259ee0fbb2298 You are receiving this because

Re: [Rpm-maint] [rpm-software-management/rpm] rpm.execute should not treat the exit status as errno number (Issue #2528)

2023-06-13 Thread Panu Matilainen
Closed #2528 as completed via #2538. -- Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/issues/2528#event-9513165486 You are receiving this because you are subscribed to this thread. Message ID: ___ Rp

Re: [Rpm-maint] [rpm-software-management/rpm] Fix error handling of rpm.execute() and rpm.redirect2null() (PR #2538)

2023-06-13 Thread Panu Matilainen
Merged #2538 into master. -- Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/pull/2538#event-9513165216 You are receiving this because you are subscribed to this thread. Message ID: ___ Rpm-maint maili

Re: [Rpm-maint] [rpm-software-management/rpm] Add %specpartsdir to macros.in (PR #2534)

2023-06-13 Thread Florian Festi
@ffesti pushed 1 commit. 0e923736841f30d20c29474c79461759b66ac519 Add %specpartsdir to macros.in -- View it on GitHub: https://github.com/rpm-software-management/rpm/pull/2534/files/ebb9d2010c9dde4e2589bfe71a88c64c1653d169..0e923736841f30d20c29474c79461759b66ac519 You are receiving this because

Re: [Rpm-maint] [rpm-software-management/rpm] License clarification (Issue #2511)

2023-06-13 Thread Panu Matilainen
Let's continue this in the [Fedora PR](https://src.fedoraproject.org/rpms/rpm/pull-request/44), other distros can and do package things differently so this could get confusing. -- Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/issues/2511#issue

Re: [Rpm-maint] [rpm-software-management/rpm] License clarification (Issue #2511)

2023-06-13 Thread Panu Matilainen
Closed #2511 as completed. -- Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/issues/2511#event-9512958985 You are receiving this because you are subscribed to this thread. Message ID: ___ Rpm-maint ma

Re: [Rpm-maint] [rpm-software-management/rpm] fix "no compatible architectures" in riscv32 (PR #2540)

2023-06-13 Thread Panu Matilainen
Thanks @rwmjones and @davidlt for the insight, I had no clue. @U2FsdGVkX1 - what's the environment you're running into this? If there's an actual use-case (distro or such) for riscv-32 now, it wouldn't hurt to have people aware of it. -- Reply to this email directly or view it on GitHub: https

Re: [Rpm-maint] [rpm-software-management/rpm] fix "no compatible architectures" in riscv32 (PR #2540)

2023-06-13 Thread David Abdurachmanov
To my knowledge none of main distributions support 32-bit RISCV. OpenEmbedde definitely has it. Most likely the same applies to Buildroot too. There are no native riscv32 hardware available that would target Linux to my knowledge. It's mainly tested with QEMU. IIRC Alibaba/T-HEAD cores are `CON

Re: [Rpm-maint] [rpm-software-management/rpm] fix "no compatible architectures" in riscv32 (PR #2540)

2023-06-13 Thread rwmjones
I asked @davidlt to take a look, but just a few things ... We don't use 32 bit RISC-V in Fedora, and don't plan to, so to some extent we don't care about it. Note that 32 bit RISC-V machines are currently confined to tiny embedded systems that couldn't possibly run Linux. eg. They lack usersp