Re: [Rpm-maint] [rpm-software-management/rpm] "load" does not work with RPM 4.16.90 (#1669)

2021-05-06 Thread Vít Ondruch
@pmatilai I must say that the error as well as your answer puzzles me. Just FTR, this is the origin of the question mark: https://src.fedoraproject.org/rpms/ruby/c/8e2921fd5a3125b7ce2487bb449b2e6db40cc3c0 IOW the question mark was always guarding the existence of the `load` macro/builtin. If

Re: [Rpm-maint] [rpm-software-management/rpm] "load" does not work with RPM 4.16.90 (#1669)

2021-05-06 Thread Jun Aruga
> Sorry could you tell me what's wrong in the following code? I was able to implement the own exists macro like this. I expect `rpmlint` skips the load. But `rpmlint` still checks the load macro printing parsing error. Now what we need to fix is rpmlint. ``` %global exists_file %{lua:

Re: [Rpm-maint] [rpm-software-management/rpm] "load" does not work with RPM 4.16.90 (#1669)

2021-05-06 Thread Jun Aruga
@pmatilai I am seeing `doc/manual/lua.md` and https://rpm.org/user_doc/lua.html . Sorry could you tell me what's wrong in the following code? ``` %load_if_exists(path) %{lua: if posix.access(arg.path, "r") then print("Loading " .. arg.path .. "..") load(arg.path) end }

Re: [Rpm-maint] [rpm-software-management/rpm] "load" does not work with RPM 4.16.90 (#1669)

2021-05-06 Thread Jun Aruga
@pmatilai ok thanks I will try it! -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/issues/1669#issuecomment-833461866___ Rpm-maint

Re: [Rpm-maint] [rpm-software-management/rpm] "load" does not work with RPM 4.16.90 (#1669)

2021-05-06 Thread Panu Matilainen
Well you can program whatever logic you want if you use %{lua:..} This behavior isn't changing back. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [Rpm-maint] [rpm-software-management/rpm] "load" does not work with RPM 4.16.90 (#1669)

2021-05-06 Thread Panu Matilainen
Closed #1669. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/issues/1669#event-4694612288___ Rpm-maint mailing list

Re: [Rpm-maint] [rpm-software-management/rpm] "load" does not work with RPM 4.16.90 (#1669)

2021-05-05 Thread Jun Aruga
Reopened #1669. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/issues/1669#event-4689242420___ Rpm-maint mailing list

Re: [Rpm-maint] [rpm-software-management/rpm] "load" does not work with RPM 4.16.90 (#1669)

2021-05-05 Thread Jun Aruga
@pmatilai when using `%{load:...}`, the rpmlint failed to part the spec file. https://github.com/rpm-software-management/rpmlint/issues/632 . The rpmlint can not detect the loaded file's existence. I see the `%{exists...}` is still new in RPM 4.17. http://rpm.org/user_doc/macros.html Is there

Re: [Rpm-maint] [rpm-software-management/rpm] "load" does not work with RPM 4.16.90 (#1669)

2021-05-05 Thread Jun Aruga
Closed #1669. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/issues/1669#event-4688478118___ Rpm-maint mailing list

Re: [Rpm-maint] [rpm-software-management/rpm] "load" does not work with RPM 4.16.90 (#1669)

2021-05-05 Thread Jun Aruga
Thanks! I confirmed it works with `%{load: ...}`. I would close this ticket. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [Rpm-maint] [rpm-software-management/rpm] "load" does not work with RPM 4.16.90 (#1669)

2021-05-05 Thread Jun Aruga
Thanks for the info! Let me check it. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/issues/1669#issuecomment-832605314___

Re: [Rpm-maint] [rpm-software-management/rpm] "load" does not work with RPM 4.16.90 (#1669)

2021-05-04 Thread Panu Matilainen
%{load:...} works as always but any undocumented special behaviors of conditionals on built-in macros are gone in 4.17, %{?foo} always tests whether said macro is defined or not, both built-ins and user-defined macros. Conditional load can be constructed with more control using expressions and

Re: [Rpm-maint] [rpm-software-management/rpm] "load" does not work with RPM 4.16.90 (#1669)

2021-05-02 Thread Jun Aruga
I can try to create a minimal reproducer if you want. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: