Re: [Rpm-maint] [rpm-software-management/rpm] Do not expand macro in a comment. (#123)

2017-01-18 Thread MyungJoo Ham
Closed #123. -- 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/123#event-928461874___ Rpm-maint mailing list

Re: [Rpm-maint] [rpm-software-management/rpm] python bindings are using system RPM instead of in-source one (#130)

2017-01-18 Thread Igor Gnatenko
I will make a patch for 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/130#issuecomment-273552868___ Rpm-maint mailing

[Rpm-maint] [rpm-software-management/rpm] rpm-python subpackage links against rpm lib on the system (#130)

2017-01-18 Thread Jan Šilhan
When I build rpm (exact 4.13 release) rpm-python subpackage links against rpm on the system, not the built one. When I tried to build rpm package providing soname 3 and rpm installed on the system had soname 7, rpm-python had requirement for soname 7 and so cannot be installed. It was

Re: [Rpm-maint] [rpm-software-management/rpm] Add OpenSSL crypto backend (#129)

2017-01-18 Thread Tomáš Mráz
t8m commented on this pull request. > +} + +int rpmDigestFinal(DIGEST_CTX ctx, void ** datap, size_t *lenp, int asAscii) +{ +int ret; +unsigned char *digest = NULL; +unsigned int digestlen; + +if (ctx == NULL) return -1; + +digestlen = EVP_MD_CTX_size(ctx->md_ctx); +

Re: [Rpm-maint] [rpm-software-management/rpm] Add OpenSSL crypto backend (#129)

2017-01-18 Thread Stephen Gallagher
sgallagher commented on this pull request. > +else { +/* ASCII requested */ +if (lenp) *lenp = (2*digestlen) + 1; +if (datap) { +const uint8_t * s = (const uint8_t *) digest; +*datap = pgpHexStr(s, digestlen); +} +} + +ret = 1;

Re: [Rpm-maint] [rpm-software-management/rpm] Add OpenSSL crypto backend (#129)

2017-01-18 Thread Stephen Gallagher
sgallagher commented on this pull request. > +WITH_OPENSSL_INCLUDE= +WITH_OPENSSL_LIB= +if test "$with_openssl" = yes; then +# If we have pkgconfig make sure CPPFLAGS are setup correctly for the OpenSSL +# -I include path. +AC_PATH_TOOL([PKGCONFIG], [pkg-config], [no],

Re: [Rpm-maint] [rpm-software-management/rpm] Add OpenSSL crypto backend (#129)

2017-01-18 Thread Stephen Gallagher
sgallagher commented on this pull request. > +} + +int rpmDigestFinal(DIGEST_CTX ctx, void ** datap, size_t *lenp, int asAscii) +{ +int ret; +unsigned char *digest = NULL; +unsigned int digestlen; + +if (ctx == NULL) return -1; + +digestlen = EVP_MD_CTX_size(ctx->md_ctx); +

Re: [Rpm-maint] [rpm-software-management/rpm] Add OpenSSL crypto backend (#129)

2017-01-18 Thread Igor Gnatenko
ignatenkobrain commented on this pull request. > +else { +/* ASCII requested */ +if (lenp) *lenp = (2*digestlen) + 1; +if (datap) { +const uint8_t * s = (const uint8_t *) digest; +*datap = pgpHexStr(s, digestlen); +} +} + +ret

Re: [Rpm-maint] [rpm-software-management/rpm] RFE: OpenSSL/LibreSSL as an alternative to NSPR/NSS or beecrypt (#119)

2017-01-18 Thread Stephen Gallagher
I just opened PR #129 Reviews and concerns are most welcome. -- 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] Add OpenSSL crypto backend (#129)

2017-01-18 Thread Stephen Gallagher
These patches add support for building RPM against OpenSSL 1.1.0 to provide hashing and signature functionality. It is enabled by passing `--with-openssl` to `configure`. (For backwards compatibility, it retains the use of `--with[out]-nss` as a toggle between beecrypt and nss). I have tested

Re: [Rpm-maint] [rpm-software-management/rpm] Support optionally creating backup files with %autopatch or %autosetup (#110)

2017-01-18 Thread Panu Matilainen
Closed #110. -- 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/110#event-927535419___ Rpm-maint mailing list

Re: [Rpm-maint] [rpm-software-management/rpm] Support optionally creating backup files with %autopatch or %autosetup (#110)

2017-01-18 Thread Panu Matilainen
Implemented as of commit 1d7b45e484883f3a340f40610b060cc100f62caa and 34c4609df37c2462dcd884e7952259b66ebd3098 as a separate "gendiff" backend. See commit messages & PR#109 comments for further rationale. Credits in the commit message already but again, thanks for input and inspiration on

Re: [Rpm-maint] [rpm-software-management/rpm] When using %autopatch, create backup files with .~ suffix by def… (#109)

2017-01-18 Thread Panu Matilainen
Closed #109. -- You are receiving this because you were mentioned. Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/pull/109#event-927531760___ Rpm-maint mailing list Rpm-maint@lists.rpm.org

Re: [Rpm-maint] [rpm-software-management/rpm] When using %autopatch, create backup files with .~ suffix by def… (#109)

2017-01-18 Thread Panu Matilainen
but the quilt backend is probably relying on quirks as it is now, need to look closer. -- You are receiving this because you were mentioned. Reply to this email directly or view it on GitHub:

Re: [Rpm-maint] [rpm-software-management/rpm] When using %autopatch, create backup files with .~ suffix by def… (#109)

2017-01-18 Thread Panu Matilainen
Oh, I know about the patches_num patch. I don't much like it, and thought ordinals aren't that helpful. Which is why there are no backups currently. I was kinda hoping you guys had come up with some nicer way to generate backups with patch numbers when I saw this ;) Anyway, on a second thought