Re: [Rpm-maint] [rpm-software-management/rpm] Python macro improvements (#221)

2017-10-23 Thread Ville Skyttä
@pmatilai @Conan-Kudo ping? -- 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/pull/221#issuecomment-338811478___ Rpm-maint mailing list

[Rpm-maint] [rpm-software-management/rpm] RFE: show command which is executed for dependency generator in debug mode (#338)

2017-10-23 Thread Igor Gnatenko
Currently I don't see way how to debug rpm dependency generators.. Something just happens silently and might be broken in some way and there is no way of finding out what's going on. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on

[Rpm-maint] [rpm-software-management/rpm] build/files: don't produce warning if file marked as %doc is already … (#337)

2017-10-23 Thread Igor Gnatenko
…included In Rust packaging there is need to have some files (license, doc) in special location (where the rest of package is) and there is no point in duplicating files. Fixes: https://github.com/rpm-software-management/rpm/issues/336 Signed-off-by: Igor Gnatenko

Re: [Rpm-maint] [rpm-software-management/rpm] RPM shows warning about including file twice when using %doc (#336)

2017-10-23 Thread Igor Gnatenko
@pmatilai obviously I made typo, it is supposed to be `%exclude /usr/share/foo/examples`... -- 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] RPM shows warning about including file twice when using %doc (#336)

2017-10-23 Thread Igor Gnatenko
``` # Generated by rust2rpm %bcond_without check %global debug_package %{nil} %global crate scopeguard Name: rust-%{crate} Version:0.3.3 Release:1%{?dist} Summary:RAII scope guard that will run a given closure when it goes out of scope License:MIT or

Re: [Rpm-maint] [rpm-software-management/rpm] tests: add tests for rich dependencies (#326)

2017-10-23 Thread ニール・ゴンパ
@pmatilai I dropped the broken tests. -- 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/pull/326#issuecomment-338646419___ Rpm-maint

Re: [Rpm-maint] [rpm-software-management/rpm] RPM shows warning about including file twice when using %doc (#336)

2017-10-23 Thread Panu Matilainen
I don't see how you'd get duplicate warnings in that scenario. An actual reproducer would help. -- 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] [WIP] tests: add tests for rich dependencies (#326)

2017-10-23 Thread ニール・ゴンパ
@pmatilai I can do that. -- 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/pull/326#issuecomment-338644484___ Rpm-maint mailing list

Re: [Rpm-maint] [rpm-software-management/rpm] [WIP] tests: add tests for rich dependencies (#326)

2017-10-23 Thread Panu Matilainen
Why not just kick the tests that dont work so we can merge the things that do? There's always time for more tests later. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

[Rpm-maint] [rpm-software-management/rpm] RPM shows warning about including file twice when using %doc (#336)

2017-10-23 Thread Igor Gnatenko
I think such construction ``` %files /usr/share/foo %doc /usr/share/examples ``` should just mark examples directory as %doc and not warn about double-inclusion. I've tried to use %exclude but it just excludes thing completely. Let me know if I can provide more details -- You are receiving

Re: [Rpm-maint] [rpm-software-management/rpm] [WIP] tests: add tests for rich dependencies (#326)

2017-10-23 Thread ニール・ゴンパ
@pmatilai A couple of the tests are failing because they test the wrong way. I haven't had a chance to fix them. I may not get to... @ignatenkobrain do you think you can finish this? -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on

Re: [Rpm-maint] [rpm-software-management/rpm] Add macro to force fsync() on close() (#187)

2017-10-23 Thread Panu Matilainen
Closed #187. -- 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/pull/187#event-1305249129___ Rpm-maint mailing list

Re: [Rpm-maint] [rpm-software-management/rpm] Add macro to force fsync() on close() (#187)

2017-10-23 Thread Panu Matilainen
Functionality included in slightly different for as of commit 4087530f0fcbb14167be8296957e44e6ffc97579, thanks for your work on this, not to mention patience. -- 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] Add RPM_CHECK_LIB to acinclude.m4 (#257)

2017-10-23 Thread Panu Matilainen
Closed #257. -- 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/257#event-1305239481___ Rpm-maint mailing list

Re: [Rpm-maint] [rpm-software-management/rpm] Add RPM_CHECK_LIB to acinclude.m4 (#257)

2017-10-23 Thread Panu Matilainen
Thanks but no thanks, a helper that's practically as long as our entire configure.ac seems like a huge overkill with the relatively few external dependencies we have. -- 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] RFC Merge RPM 4.13 OS/2 code changes (#260)

2017-10-23 Thread Panu Matilainen
Sorry but there's just no way we can justify this amount of added code + complexity = maintenance burden for a long dead OS. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: