Closed #3850 as completed via #3857.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/3850#event-18751813636
You are receiving this because you are subscribed to this thread.
Message ID:
___
R
Merged #3857 into master.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3857#event-18751813295
You are receiving this because you are subscribed to this thread.
Message ID:
___
Rpm-maint mail
@ffesti pushed 1 commit.
45d636a8553f2f3c63605ad23ff10f264be285fc Merge branch 'master' into
mktree-toolbox
--
View it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3857/files/c26e7cad2b1259e3365e4427b880242add986cb9..45d636a8553f2f3c63605ad23ff10f264be285fc
You are receiving
Merged #3863 into master.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3863#event-18751351203
You are receiving this because you are subscribed to this thread.
Message ID:
___
Rpm-maint mail
ffesti left a comment (rpm-software-management/rpm#3860)
Thanks for the patch!
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3860#issuecomment-3102503448
You are receiving this because you are subscribed to this thread.
Message ID: ___
Merged #3860 into master.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3860#event-18751212179
You are receiving this because you are subscribed to this thread.
Message ID:
___
Rpm-maint mail
Yes, this can happen if you are using this particular tags. There are a few
places that break things intentionally. That's what it means to update the
major version number (although I understand people can't remember the last time
when it changed from 3 to 4).
--
Reply to this email directly o
ffesti left a comment (rpm-software-management/rpm#3859)
Yeah, the debuginfo code does not play that well with others. This needs to be
fixed.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/3859#issuecomment-3048108027
You are receivi
ffesti left a comment (rpm-software-management/rpm#3841)
Thanks a lot for your work!
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3841#issuecomment-3012640976
You are receiving this because you are subscribed to this thread.
Message I
Merged #3841 into master.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3841#event-18355347187
You are receiving this because you are subscribed to this thread.
Message ID:
___
Rpm-maint mail
@ffesti commented on this pull request.
> +if (move_rpmrc && fs::is_symlink(oldrpmrc, ec2))
+ fs::remove(oldrpmrc, ec2);
+
+if (fs::is_regular_file(newmacros, ec2))
+ fs::rename(newmacros, oldmacros, ec2);
+if (fs::is_regular_file(newrpmrc, ec2))
+ fs::rename(newrpm
@ffesti commented on this pull request.
> @@ -38,9 +38,12 @@ _macro path_ uses this to achieve the following hierarchy
> of settings:
The default _macro path_ can be inspected with *rpm --showrc|grep ^Macro*.
-In older versions of rpm, the path of per-user macros was _~/.rpmmacros_.
-This
@ffesti commented on this pull request.
> +fs::path newrpmrc = userdir_path / "rpmrc";
+
+if (!isatty(0)) /* only on interactive runs */
+ return -1;
+
+if (!fs::is_regular_file(oldmacros) && !fs::is_regular_file(oldrpmrc))
+ return -1;
+
+int fd = open(lockfile.c_str
@ffesti pushed 1 commit.
a7b1728c655b9d3554c3ddf433fa0caffeba2136 Move ~/.rpmmacros and ~/.rpmrc to
~/.config/rpm/
--
View it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3750/files/9ab1dd26dced69314715472f20b56fd46c8f6c66..a7b1728c655b9d3554c3ddf433fa0caffeba2136
You are rec
ffesti left a comment (rpm-software-management/rpm#3750)
OK, as we no longer copy any files we really shouldn't delete anything but the
symlinks we created. As said above the test suite doesn't like chattr (even
when installing it in the container).
This no longer does the migration with root p
@ffesti pushed 1 commit.
9ab1dd26dced69314715472f20b56fd46c8f6c66 Move ~/.rpmmacros and ~/.rpmrc to
~/.config/rpm/
--
View it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3750/files/0f50b6c7dad42ad5ed5ff9460a7d60a2ca4f7f95..9ab1dd26dced69314715472f20b56fd46c8f6c66
You are rec
ffesti left a comment (rpm-software-management/rpm#3840)
And this, kids, is why we don't push directly onto master...
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3840#issuecomment-3008521027
You are receiving this because you are subs
@ffesti pushed 1 commit.
11dc35e61f85961f4dcb8614e757946c2b97011e Fix doubled word in man page
--
View it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3840/files/fa6a6b81a075870110268263452860d1319c7bf9..11dc35e61f85961f4dcb8614e757946c2b97011e
You are receiving this because y
@ffesti pushed 1 commit.
0f50b6c7dad42ad5ed5ff9460a7d60a2ca4f7f95 Move ~/.rpmmacros and ~/.rpmrc to
~/.config/rpm/
--
View it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3750/files/6fade43310a6fa4e43243c5b7bbcc44347b2658a..0f50b6c7dad42ad5ed5ff9460a7d60a2ca4f7f95
You are rec
You can view, comment on, or merge this pull request online at:
https://github.com/rpm-software-management/rpm/pull/3840
-- Commit Summary --
* Fix doubled word in man page
-- File Changes --
M docs/man/rpm-config.5.scd (2)
M po (2)
-- Patch Links --
https://github.com/rpm-softw
Closed #3815 as completed via #3834.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/3815#event-18330396550
You are receiving this because you are subscribed to this thread.
Message ID:
___
R
Merged #3834 into master.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3834#event-18330396181
You are receiving this because you are subscribed to this thread.
Message ID:
___
Rpm-maint mail
Closed #3817 as completed via 927367b8e2fb67cd7f5a639d386e068c685a02ac.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/3817#event-18330249445
You are receiving this because you are subscribed to this thread.
Message ID:
__
Merged #3832 into master.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3832#event-18330249121
You are receiving this because you are subscribed to this thread.
Message ID:
___
Rpm-maint mail
ffesti left a comment (rpm-software-management/rpm#3750)
> An easy way to force the above scenario is 'chattr +i ~/.rpmmrc'.
>
> It's the good old "the case you don't test is the one with the bugs" 😅
What is super annoying about this is that I had tests using `chattr +i` for
both files separate
ffesti left a comment (rpm-software-management/rpm#3762)
OK, turns out this is caused somewhere between
c00d8a0ed32169baf82cd0a34ffd1c9377b09c38 (last good) and
08a6a5e848f007c2e8d83fc1df8712d1a43229f4 (know broken). So during converting
the finger printing code to C++. Some intermediate patche
Merged #3824 into master.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3824#event-18266657245
You are receiving this because you are subscribed to this thread.
Message ID:
___
Rpm-maint mail
Merged #3809 into master.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3809#event-18266769280
You are receiving this because you are subscribed to this thread.
Message ID:
___
Rpm-maint mail
Closed #3785 as completed via #3809.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/3785#event-18266769533
You are receiving this because you are subscribed to this thread.
Message ID:
___
R
ffesti left a comment (rpm-software-management/rpm#3819)
Uff, this is a lot of test suite work "just" for changing a few return codes.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3819#issuecomment-2990506382
You are receiving this bec
Merged #3819 into master.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3819#event-18243187444
You are receiving this because you are subscribed to this thread.
Message ID:
___
Rpm-maint mail
Closed #2581 as completed via #3819.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/2581#event-18243187739
You are receiving this because you are subscribed to this thread.
Message ID:
___
R
@ffesti requested changes on this pull request.
While the lists of functions look good the explaining in between is still a bit
thin. Yes, this isn't a tutorial but people need to be able to pick up on what
this is about.
> @@ -0,0 +1,960 @@
+RPM-LUA(7)
+
+# NAME
+
+rpm-lua - RPM embedded Lua i
@ffesti converted this issue into discussion #3818.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/3810#event-18188491298
You are receiving this because you are subscribed to this thread.
Message ID:
__
Merged #3814 into master.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3814#event-18188195269
You are receiving this because you are subscribed to this thread.
Message ID:
___
Rpm-maint mail
ffesti left a comment (rpm-software-management/rpm#3810)
RPM 4.15.1 was released on Nov 18 2019. You may want to try a more recent
version.
You can also try to get a clue what's happening by looking what processes are
running at the time or strace the rpmbuild process if that one is hugging the
ffesti left a comment (rpm-software-management/rpm#3810)
What version of rpm do you use? May be it is still missing some of the recent
performance improvements
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/3810#issuecomment-297951064
ffesti left a comment (rpm-software-management/rpm#3810)
> But you probably shouldn't as the test is there for a reason.
Well, you should at least once to make sure this is actually what's taking up
the hour.
You can also `%undefine __check_files` in the spec files to disable the check
for the
ffesti left a comment (rpm-software-management/rpm#3810)
As stated in the macros file (typically /usr/lib/rpm/macros) you can just
comment out `%__check_files` there to disable the check. But you probably
shouldn't as the test is there for a reason.
But this taking a full hour is obviously a pr
@ffesti commented on this pull request.
> @@ -554,6 +524,28 @@ unique to verify mode are:
| *P*
: ca*P*abilities differ
+|[ *Attribute*
+:< *Description*
+| *a*
+: *%artifact* - an implicit side-effect file (eg. build-id links)
+| *c*
+: *%config*uration file
+| *d*
+: *%doc*document
ffesti left a comment (rpm-software-management/rpm#3807)
Looks good. The Python tests are only run if PYTHON is set.
Thanks for the patch!
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3807#issuecomment-2969850242
You are receiving thi
Merged #3807 into master.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3807#event-18131203950
You are receiving this because you are subscribed to this thread.
Message ID:
___
Rpm-maint mail
Closed #3642 as completed via a396f10d62d22a67160e6c01bc96556d93acbf11.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/3642#event-18108679548
You are receiving this because you are subscribed to this thread.
Message ID:
__
Merged #3798 into master.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3798#event-18108679072
You are receiving this because you are subscribed to this thread.
Message ID:
___
Rpm-maint mail
@ffesti commented on this pull request.
> @@ -69,6 +69,8 @@ runroot rpmkeys -Kv --nosignature
> /data/RPMS/hello-1.0-1.i386.rpm
Header SHA1 digest: NOTFOUND
Payload SHA256 digest: NOTFOUND
Payload SHA256 ALT digest: NOTFOUND
+Payload SHA3-256 digest: NOTFOUND
I wonder if we
Merged #3787 into master.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3787#event-18071465763
You are receiving this because you are subscribed to this thread.
Message ID:
___
Rpm-maint mail
ffesti left a comment (rpm-software-management/rpm#3781)
I wonder if the **EXPANSION** section should be before *BUILT-IN MACROS*.
Otherwise the expansion syntax is only in the **SYNOPSIS** for all the examples
of the build-in macros.
The examples in *BUILT-IN MACROS mix `%{name:arg}` and `%{na
@ffesti commented on this pull request.
> +specified by *%myprog* with it's first argument and always passing
+the option *--some-opt* to it, and additionally the *--xtra* option
+if it received the *-x* option itself:
+
+```
+%define myhelper(x) %{myprog} --some-opt %{?-x:--xtra} %{1}
+```
+
+#
@ffesti commented on this pull request.
> +Some useful tools for working with and troubleshooting macros:
+
+*rpm --eval "*_EXPRESSION_*"*
+ Expand macro _EXPRESSION_ on the command line.
+
+*rpm --define "aa 11" --eval "%aa"*
+ Define and evalue a macro on the command line.
+
+*rpml
@ffesti commented on this pull request.
> +```
+%[ "%{aa}" == "5" ? 1 : 2]
+```
+
+## Example 7. Define a parametric Lua macro
+Define *%range* Lua macro that takes two integers as arguments and
+expands to the catenation of the numbers in the range:
+
+```
+%define range() %{lua:
+for i=mat
@ffesti commented on this pull request.
> +
+*%{span:*_STRING_*}*
+ As-is string, handy for wrapping multiline macros.
+ Since 6.0.0.
+
+ Example:
+ ```
+ %{span:
+ %one thing
+ %another thing
+ }
+ ```
+
+*%{sub:*_STRING_, _I
Closed #2373 as completed via #3776.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/2373#event-18069835564
You are receiving this because you are subscribed to this thread.
Message ID:
___
R
Merged #3776 into master.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3776#event-18069835229
You are receiving this because you are subscribed to this thread.
Message ID:
___
Rpm-maint mail
@ffesti pushed 1 commit.
0266c6199a0e87737a353073c5ffe6cc48696443 Move ~/.rpmmacros and ~/.rpmrc to
~/.config/rpm/
--
View it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3750/files/dbedb24d6c99951669eff53b9c83f0838c87270c..0266c6199a0e87737a353073c5ffe6cc48696443
You are rec
ffesti left a comment (rpm-software-management/rpm#3715)
Converted to draft to get it out of our review lists until it is actual time to
merge.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3715#issuecomment-2943836920
You are receivin
Merged #3777 into master.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3777#event-17979537715
You are receiving this because you are subscribed to this thread.
Message ID:
___
Rpm-maint mail
ffesti left a comment (rpm-software-management/rpm#3775)
Yes, this is clearly the behavior we do want. For now this really only applies
to file names but this is less than helpful.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/3775#i
@ffesti pushed 15 commits.
dc420cf60fb7a25d25d0af5b3e989bf4de447107 Add support for multiple OpenPGP
signatures per package, part 1/2
5adbed27b6862f9103f8552e6287fdb8dace0d28 Add support for multiple OpenPGP
signatures per package, part 2/2
5bc5f60fb0b6d61fb4818ea8dd99a383bf7b6a22 Adjust sign
@ffesti pushed 17 commits.
cb1d8f7ee9ddb780716d2948cc8a99d62659e07a Add a new unique v6-only reserved
signature tag below header base
1ee45b836f47b8e7d6c49892ae5a0458fc757721 Add tag extension for rpm format
version detection
c51a67d584532dff8e97dd505cacf94dc7c0188f Add support for multiple O
@ffesti approved this pull request.
Looks good now. Needs "squash and merge".
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3763#pullrequestreview-2870412687
You are receiving this because you are subscribed to this thread.
Message I
Closed #3682 as completed via d8de849e02aadb6421d3492b725bf21b3c771bdd.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/3682#event-17825801162
You are receiving this because you are subscribed to this thread.
Message ID:
__
@ffesti requested changes on this pull request.
Fixed minor issues directly. A few places could be a bit more clear (see
comments).
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3763#pullrequestreview-2870120938
You are receiving thi
@ffesti commented on this pull request.
> + A hard limit for maximum number of threads to use in parallel
+ during a package build. Zero means unlimited.
+
+*%\_smp_tasksize_proc* _NUMBER_
+ Assumed task size of build processes (during compilation).
+ Used for tuning the am
Closed #3682 as completed via #3771.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/3682#event-17825801121
You are receiving this because you are subscribed to this thread.
Message ID:
___
R
Merged #3771 into master.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3771#event-17825800841
You are receiving this because you are subscribed to this thread.
Message ID:
___
Rpm-maint mail
@ffesti commented on this pull request.
> +*%packager* _STRING_
+ Configurable packager information, same as Packager: in a specfile.
+
+*%translationurl* _URL_
+ Configurable translation URL, same as TranslationURL: in a specfile.
+
+*%upstreamreleases* _URL_
+ Configurable UR
@ffesti pushed 1 commit.
4b446676a8976a1304477e6d3ca8376ebc918a7a Remove duplicated section
--
View it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3763/files/6e945ed5bf4b3e10f331f5d66834c67a177063f6..4b446676a8976a1304477e6d3ca8376ebc918a7a
You are receiving this because you
@ffesti pushed 1 commit.
6e945ed5bf4b3e10f331f5d66834c67a177063f6 Drop spurious "the"
--
View it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3763/files/276ea01fdb4e02e6f16cafe6ad04fb0259952e96..6e945ed5bf4b3e10f331f5d66834c67a177063f6
You are receiving this because you are su
@ffesti pushed 1 commit.
276ea01fdb4e02e6f16cafe6ad04fb0259952e96 Fix typo rpmbuild-config.5.scd
--
View it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3763/files/45aceb1d2149a97f37e9e6532d54510eab30d890..276ea01fdb4e02e6f16cafe6ad04fb0259952e96
You are receiving this because
@ffesti pushed 21 commits.
383eb0fc371d65c9c3c507627a3feea50cd8955c Add macros.rpmsign-sequoia to doc dir
ffc047392811837e46215047e1665f69cb32080a Add sequoia-sq to test suite
c02b343f9766ba247140e62d4680355aba0edbdd Drop unused header arguments, update
comments in signing internals
d070a2d9c8
Merged #3769 into master.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3769#event-17812533780
You are receiving this because you are subscribed to this thread.
Message ID:
___
Rpm-maint mail
ffesti left a comment (rpm-software-management/rpm#3760)
Integration
---
Magika is written is Rust and currently has bindings for Go, JavaScript and
Python. In theory Rust libraries can be used from C/C++ but they currently
don't offer the necessary wrappers and headers. This is probabl
Closed #3752 as completed via ad114b0174c26fa101ce9bbf82930b2c2e421a09.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/3752#event-17715876060
You are receiving this because you are subscribed to this thread.
Message ID:
__
ffesti left a comment (rpm-software-management/rpm#3757)
LGTM. Took a while to see how this `sigtd->tag = pgpDigParamsAlgo()` trick work
for multiple signatures. But ofc we don't use different tags for that at all
and compare the actual signatures instead of tags.
--
Reply to this email direct
Merged #3757 into master.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3757#event-17715875614
You are receiving this because you are subscribed to this thread.
Message ID:
___
Rpm-maint mail
@ffesti pushed 1 commit.
dbedb24d6c99951669eff53b9c83f0838c87270c Move ~/.rpmmacros and ~/.rpmrc to
~/.config/rpm/
--
View it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3750/files/b078ebe1ad7e004d13a5ad0e1c766b49c40f69a9..dbedb24d6c99951669eff53b9c83f0838c87270c
You are rec
ffesti left a comment (rpm-software-management/rpm#3750)
OK, cleaning up is a bit of a mess (no pun intended). The new code is pretty
ugly but as we need to undo both files if one fails and remove the directory
afterwards. So I just intertwined both operations manually.
I don't think we really
ffesti left a comment (rpm-software-management/rpm#3750)
> Actually so I think I understand the question now. We should definitely check
> what `XDG_CONFIG_HOME` is set to and fallback if it is unset.
We actually do that in the surrounding code already. It's just not part of this
particular pat
The XDG cofiguration directory is the new default location for the macros and
rpmrc file. Migrate legacy files automatically. Create symlinks from the home
directory for compatibility with older rpm versions.
Resolves: #3467
You can view, comment on, or merge this pull request online at:
http
@ffesti pushed 1 commit.
b078ebe1ad7e004d13a5ad0e1c766b49c40f69a9 Move ~/.rpmmacros and ~/.rpmrc to
~/.config/rpm/
--
View it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3750/files/91d5dbb330b30ddc32d507cb2b3a0a24de167ba3..b078ebe1ad7e004d13a5ad0e1c766b49c40f69a9
You are rec
ffesti left a comment (rpm-software-management/rpm#3750)
I wonder if we want to do the move if XDG points to a non standard place.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3750#issuecomment-2875650298
You are receiving this because
ffesti left a comment (rpm-software-management/rpm#3746)
Guess technically gcc does compile just fine. I looks like the binarys are
linked against the wrong library.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/3746#issuecomment-285
ffesti left a comment (rpm-software-management/rpm#3747)
Thanks a lot!
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3747#issuecomment-2854474537
You are receiving this because you are subscribed to this thread.
Message ID: ___
Closed #3746 as completed via #3747.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/3746#event-17536981503
You are receiving this because you are subscribed to this thread.
Message ID:
___
R
Merged #3747 into master.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3747#event-17536981198
You are receiving this because you are subscribed to this thread.
Message ID:
___
Rpm-maint mail
Created #3748 as this seems very much a legitimate improvement.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/discussions/3724#discussioncomment-13049958
You are receiving this because you are subscribed to this thread.
Message ID:
ffesti created an issue (rpm-software-management/rpm#3748)
### Discussed in https://github.com/rpm-software-management/rpm/discussions/3724
Originally posted by **scfc** April 18, 2025
The output of dependency generators is filtered by several macros acting as
regular expressions either on fi
ffesti left a comment (rpm-software-management/rpm#3746)
The gcc man page says `When you compile C++ programs, you should invoke GCC as
g++ instead.`
Sorry, didn't catch that either.
Guess the INSTALL file should talk about C++ a bit more clearly (like at all).
--
Reply to this email directl
ffesti left a comment (rpm-software-management/rpm#3744)
Thanks a lot for the patches!
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3744#issuecomment-2853683354
You are receiving this because you are subscribed to this thread.
Message
Merged #3744 into master.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3744#event-17532727532
You are receiving this because you are subscribed to this thread.
Message ID:
___
Rpm-maint mail
ffesti left a comment (rpm-software-management/rpm#3746)
Here is the script I used to build:
```
#!/bin/sh
export CXXFLAGS="-Og -g"
export CFLAGS="${CXXFLAGS}"
cmake \
-DCMAKE_INSTALL_PREFIX=/usr \
-DENABLE_WERROR=ON \
-DENABLE_ASAN=ON \
-DENABLE_UBSAN=ON \
ffesti left a comment (rpm-software-management/rpm#3746)
RPM master requires C++ 20 and states so in CMakeLists.txt:
```
set(CMAKE_CXX_STANDARD 20)
set(CMAKE_CXX_STANDARD_REQUIRED ON)
```
These error messages look like a C++ standard before 17 is used for some reason
(e.g. `std::filesystem::re
ffesti left a comment (rpm-software-management/rpm#3740)
A quick Google search suggests adding `export GPG_TTY=$(tty)`. Can you try
this, please?
Not sure if this will really works in the GH Action environment, but it surely
is worth a try.
--
Reply to this email directly or view it on GitHub
ffesti left a comment (rpm-software-management/rpm#3738)
Added a PR with the proposed fixes for some issues. Here the comments on
everything else:
> Man page: rpm2archive.8
> Issue:Remove full stop
>
> "rpm2archive - Create tar archive from RPM Package Manager (RPM) package."
Already fi
These have been reported by the manpage-l10n project
(https://manpages-l10n-team.pages.debian.net/manpages-l10n/)
Thanks to https://github.com/hmartink for doing so!
These reported issues were fixed:
Man page: rpm2archive.8
Issue 1: \\f[B]rpm2cpio\\f[R] → \\f[B]rpm2cpio\\f[R](8)
Issue 2: \\f[
Merged #3729 into master.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3729#event-17367907930
You are receiving this because you are subscribed to this thread.
Message ID:
___
Rpm-maint mail
@ffesti pushed 1 commit.
e0e769b7e5efa9f0f3b52b81faf473b880a37061 Update dbiset.hh
--
View it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3729/files/41729aa058a0346798584f150ded70ee31c64f61..e0e769b7e5efa9f0f3b52b81faf473b880a37061
You are receiving this because you are subsc
@ffesti pushed 1 commit.
41729aa058a0346798584f150ded70ee31c64f61 Remove leftover "is"
--
View it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3729/files/811590f0aa1ee3ebbb27fd383eeb4e607cc95755..41729aa058a0346798584f150ded70ee31c64f61
You are receiving this because you are s
ffesti left a comment (rpm-software-management/rpm#3729)
there is still an left over "is" in the first docstring.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3729#issuecomment-2823715341
You are receiving this because you are subscrib
ffesti left a comment (rpm-software-management/rpm#3729)
Test case does indeed fail without the fix and passes afterwards.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3729#issuecomment-2823678192
You are receiving this because you are
1 - 100 of 1571 matches
Mail list logo
