[Rpm-maint] [rpm-software-management/rpm] Conditional macro expansion fails (#127)

2017-01-13 Thread Andreas Scherer
I have [at least one example](https://github.com/ascherer/sgb/blob/local/sgb.spec#L53) where the conditional macro `%{?with_patches:-p1}` does _not_ produce the expected `-p1` option correctly. This occurs with **rpmbuild 4.12** on **Kubuntu 16.04**. -- You are receiving this because you are

Re: [Rpm-maint] [rpm-software-management/rpm] %include breaks if space is included in path (#125)

2017-01-13 Thread Chenxiong Qi
@Conan-Kudo No. It doesn't work. Same error. -- 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/125#issuecomment-272600896___ Rp

Re: [Rpm-maint] [rpm-software-management/rpm] %include breaks if space is included in path (#125)

2017-01-13 Thread ニール・ゴンパ
Does doing the following work? ```python rpm.expandMacro('%define _sourcedir "/home/cqi/my\ package/systemd/"') ``` -- 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/125#is

Re: [Rpm-maint] [rpm-software-management/rpm] [WIP] Replace gzip/bzip2/xz with pigz/pbzip2/pxz for multicore (#126)

2017-01-13 Thread proyvind
As the multithreaded (de)compression support for pigz isn't implemented in the binary, the benefits from aren't really there as it won't be possible to implement multithreaded (de)compression for using in the rpmio library. Same goes for pbzip2. For xz, multithreaded compression support has been

Re: [Rpm-maint] [rpm-software-management/rpm] Support pbzip command for multicore environment (#117)

2017-01-13 Thread Geunsik Lim
Closed #117. -- 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/117#event-922142667___ Rpm-maint mailing list Rpm-maint@lists.rpm.or

Re: [Rpm-maint] [rpm-software-management/rpm] Support pbzip command for multicore environment (#117)

2017-01-13 Thread Geunsik Lim
I have being enabled the pigz|pbzip2 in foo.spec files (approximately 800+ packages) to decompress .tar.gz|.tar.bz2 files in parallel as following: **1st step)** for decompressing ./SOURCES/*.tar.gz|bz2 files in parallel (with alias or with symbolic) alias tar='tar --use-compress-program=pbzip2

Re: [Rpm-maint] [rpm-software-management/rpm] [WIP] Replace gzip/bzip2/xz with pigz/pbzip2/pxz for multicore (#126)

2017-01-13 Thread Geunsik Lim
Closed #126. -- 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/126#event-922126663___ Rpm-maint mailing list Rpm-maint@lists.rpm.or

Re: [Rpm-maint] [rpm-software-management/rpm] [WIP] Replace gzip/bzip2/xz with pigz/pbzip2/pxz for multicore (#126)

2017-01-13 Thread Geunsik Lim
@pmatilai , Sorry for the inconvenience caused to you. I will propose the a PR after finding the complete solution to support xz as well as gzip/bzip2. This WIP will be closed. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitH

Re: [Rpm-maint] [rpm-software-management/rpm] Error if a macro in a comment is expanded to multiple lines (#124)

2017-01-13 Thread Panu Matilainen
pmatilai commented on this pull request. rpmlog(RPMLOG_ERR, ...) does not stop the build any more than the rpmlog(RPMLO_WARNING, ...) does a few lines below. As you should've noticed when testing it. Another possible (not saying it's good or what I want, just enumerating) approach would be add

Re: [Rpm-maint] [rpm-software-management/rpm] [WIP] Replace gzip/bzip2/xz with pigz/pbzip2/pxz for multicore (#126)

2017-01-13 Thread Dmitry V. Levin
ldv-alt requested changes on this pull request. The idea is wrong: you cannot *replace* gzip, bzip2, and xz because they are much more widespread. The implementation is wrong: it does not achieve the declared goal. -- You are receiving this because you are subscribed to this thread. Reply t

Re: [Rpm-maint] [rpm-software-management/rpm] [WIP] Replace gzip/bzip2/xz with pigz/pbzip2/pxz for multicore (#126)

2017-01-13 Thread Panu Matilainen
After wasting time looking at the obviously completely untested patch against an imaginary library in PR #117 ... to put it really really politely, I'm not in a very positive mood towards this work. I'm not going to look at a single patch until you show me some numbers to back up the change, fo

[Rpm-maint] [rpm-software-management/rpm] [WIP] Replace gzip/bzip2/xz with pigz/pbzip2/pxz for multicore (#126)

2017-01-13 Thread Geunsik Lim
…nment This PR is to fix issue #113. And It can be used as an alternative of PR #117. In modern generation, most of the developers are using a machine based on multi-core architecture. From now on, Let's prepare to support parallelism. pigz is compatible with the existing gzip. It means that pigz