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

2023-03-22 Thread Daniel Alley
Filed https://github.com/rpm-software-management/rpm/issues/2447 for updating PAYLOADFORMAT -- Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/discussions/2374#discussioncomment-5400068 You are receiving this because you are subscribed to this

[Rpm-maint] [rpm-software-management/rpm] Pick a new string to be used for PAYLOADFORMAT, and support it properly (Issue #2447)

2023-03-22 Thread Daniel Alley
RPMs produced since 4.12 use a [stripped down payload format](https://github.com/rpm-software-management/rpm/pull/2433) based on CPIO, but not compatible with CPIO. However this isn't currently reflected by the handling of the `PAYLOADFORMAT` tag, which always writes `cpio`.

Re: [Rpm-maint] [rpm-software-management/rpm] Consider available memory and address space for parallel execution (PR #2418)

2023-03-22 Thread Michal Domonkos
Other than the above, the overall changeset looks sane to me. -- Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/pull/2418#issuecomment-1479932861 You are receiving this because you are subscribed to this thread. Message ID:

Re: [Rpm-maint] [rpm-software-management/rpm] Consider available memory and address space for parallel execution (PR #2418)

2023-03-22 Thread Michal Domonkos
@dmnks commented on this pull request. > +long int pagesize = sysconf(_SC_PAGESIZE); +long int pages = sysconf(_SC_PHYS_PAGES); + +if (pagesize < 0) + pagesize = 4096; +if (pages > 0) + mem = pages * pagesize; + +return mem; +} + +static unsigned long

Re: [Rpm-maint] [rpm-software-management/rpm] Consider available memory and address space for parallel execution (PR #2418)

2023-03-22 Thread Michal Domonkos
@dmnks commented on this pull request. > @@ -1174,6 +1177,89 @@ static void doShescape(MacroBuf mb, rpmMacroEntry me, > ARGV_t argv, size_t *parse mbAppend(mb, '\''); } +static unsigned long getmem_total(void) +{ +unsigned long mem = 0; +long int pagesize =

Re: [Rpm-maint] [rpm-software-management/rpm] Consider available memory and address space for parallel execution (PR #2418)

2023-03-22 Thread Michal Domonkos
@dmnks commented on this pull request. > @@ -1174,6 +1177,89 @@ static void doShescape(MacroBuf mb, rpmMacroEntry me, > ARGV_t argv, size_t *parse mbAppend(mb, '\''); } +static unsigned long getmem_total(void) +{ +unsigned long mem = 0; +long int pagesize =

Re: [Rpm-maint] [rpm-software-management/rpm] Consider available memory and address space for parallel execution (PR #2418)

2023-03-22 Thread Michal Domonkos
@dmnks commented on this pull request. > @@ -1174,6 +1177,89 @@ static void doShescape(MacroBuf mb, rpmMacroEntry me, > ARGV_t argv, size_t *parse mbAppend(mb, '\''); } +static unsigned long getmem_total(void) +{ +unsigned long mem = 0; +long int pagesize =

Re: [Rpm-maint] [rpm-software-management/rpm] Consider available memory and address space for parallel execution (PR #2418)

2023-03-22 Thread Michal Domonkos
@dmnks commented on this pull request. > + * Conservative estimates for thread use on 32bit systems where address + * space is an issue: 2GB for bare metal, 3GB for a 32bit process + * on a 64bit system. + */ +if (thread) { + unsigned long vmem = mem; +#if __WORDSIZE

Re: [Rpm-maint] [rpm-software-management/rpm] Consider available memory and address space for parallel execution (PR #2418)

2023-03-22 Thread Michal Domonkos
@dmnks commented on this pull request. > + * Conservative estimates for thread use on 32bit systems where address + * space is an issue: 2GB for bare metal, 3GB for a 32bit process + * on a 64bit system. + */ +if (thread) { + unsigned long vmem = mem; +#if __WORDSIZE

[Rpm-maint] [rpm-software-management/rpm] Accessing macro parameter from lua fails with "attempt to index a nil value (global 'opt')" (Issue #2446)

2023-03-22 Thread Ralf Habacker
The documentation at https://github.com/rpm-software-management/rpm/blob/master/docs/manual/lua.md mentions how to get macro options as ``` %foo(a:b) %{lua: if opt.b then ... ``` but this does not work on openSUSE Leap 15.4 with rpm with version: ``` rpm --version RPM version 4.14.3 ```

[Rpm-maint] [rpm-software-management/rpm] Add documentation for shell like macro '%**' (PR #2445)

2023-03-22 Thread Ralf Habacker
See https://github.com/rpm-software-management/rpm/issues/2444 You can view, comment on, or merge this pull request online at: https://github.com/rpm-software-management/rpm/pull/2445 -- Commit Summary -- * Add documentation for shell like macro %** -- File Changes -- M

[Rpm-maint] [rpm-software-management/rpm] Documentation does not mention %** to get all macro parameter (Issue #2444)

2023-03-22 Thread Ralf Habacker
At https://github.com/rpm-software-management/rpm/blob/7b0b5e7d2cf53817ec5e2545c5f591a26db72cec/docs/manual/macros.md there is mentioned ``` %* all arguments (unlike shell, not including any processed flags) ``` but not ``` %** all arguments (including any processed flags) ``` which

Re: [Rpm-maint] [rpm-software-management/rpm] Create content handler plugin hook (PR #2416)

2023-03-22 Thread Richard Phibel
@pmatilai @DemiMarie Did you get a chance to have a look at this? -- Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/pull/2416#issuecomment-1479478779 You are receiving this because you are subscribed to this thread. Message ID:

[Rpm-maint] [rpm-software-management/rpm] Spec file versioning macros (Issue #2443)

2023-03-22 Thread Cristian Le
This is a reference from fedora packaging committee [issue](https://pagure.io/packaging-committee/issue/1264): > I would like to propose a few helper macros for dealing with version control: > - Format git tag -> rpm version: e.g. 1.2.3-rc1/1.2.3rc -> 1.2.3~rc1. This > could be detected from

Re: [Rpm-maint] [rpm-software-management/rpm] rpmds related cleanups + sanity fixes (PR #2442)

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

Re: [Rpm-maint] [rpm-software-management/rpm] rpmds related cleanups + sanity fixes (PR #2442)

2023-03-22 Thread Panu Matilainen
@pmatilai pushed 1 commit. 3dc86c54c7f0af5afa6bf8ebbf2d2c155dc1f777 Eliminate code duplication from rpmdsNext() -- View it on GitHub: https://github.com/rpm-software-management/rpm/pull/2442/files/fb63b172a4aed67a46750aae8aa88747c6584f23..3dc86c54c7f0af5afa6bf8ebbf2d2c155dc1f777 You are

Re: [Rpm-maint] [rpm-software-management/rpm] rpmds related cleanups + sanity fixes (PR #2442)

2023-03-22 Thread Panu Matilainen
@pmatilai pushed 2 commits. e7d07e47ed2c9d3170086a9617c131ad95578936 Kick out one last remaining nopromote remnant from the ABI fb63b172a4aed67a46750aae8aa88747c6584f23 Kick out rpmds debugging remnants -- View it on GitHub:

[Rpm-maint] [rpm-software-management/rpm] rpmds related cleanups + sanity fixes (PR #2442)

2023-03-22 Thread Panu Matilainen
See commits for details. You can view, comment on, or merge this pull request online at: https://github.com/rpm-software-management/rpm/pull/2442 -- Commit Summary -- * Clean up rpmteColorDS(), support all dependency types * Clean up filedepTag(), support all dependency types (in theory)