Re: [Rpm-maint] Pruning self-dependencies?

2007-07-04 Thread Ville Skyttä
On Monday 02 July 2007, Adam Jackson wrote: On Mon, 2007-07-02 at 21:50 +0300, Ville Skyttä wrote: As for how many dependencies this would eliminate, running some quick queries [0] against the Fedora primary sqlite metadata database told me it'd be about 7.3% of all dependencies (9246

Re: [Rpm-maint] changelog truncating

2007-08-30 Thread Ville Skyttä
On Thursday 30 August 2007, seth vidal wrote: Hi, Would it be possible to add an option to the specfile parser or to rpmbuild that would tell it to truncate the changelogs included in the package last N entries? It would default to including them all but give us the option of clipping off

[Rpm-maint] find-lang.sh improvements

2007-10-13 Thread Ville Skyttä
Hello, Attached is a patch for find-lang.sh containing the following improvements: - spelling fixes - POSIX compliance fix for find(1) usage - Qt translation support - localized man page support - match *.omf, not *omf Split patches available at http://scop.fedorapeople.org/patches/rpm/ if you

Re: [Rpm-maint] Problems with %Requires(preun):

2007-12-11 Thread Ville Skyttä
On Tuesday 11 December 2007, Scott Bambrough wrote: Pixel wrote: afaik rpm does not order package removal :-( Interesting. The section on context marked dependencies in the snapshot version of Maximum RPM suggests it should. [...] Is their a disconnect between the documentation and the

Re: [Rpm-maint] allow to remove Requires(pre) or not?

2008-02-27 Thread Ville Skyttä
On Tuesday 26 February 2008, Panu Matilainen wrote: On Sat, 23 Feb 2008, Pixel wrote: as for me i'm not convinced that Requires(pre) not implying Requires is a feature. I would be in favor of Requires(xxx) implies Requires. Agreed, permitting remove of (pre|post|...)-only dependencies

Re: [Rpm-maint] RPM Managment system.

2008-07-07 Thread Ville Skyttä
On Monday 07 July 2008, Stanislav Brabec wrote: Nesreen wrote: 1- I wander that upon making a package using RPM software, can I assign version/build numbers for each file included in my package , No, you cannot. Version/release are assigned only for packages. But you can have one file per

Re: [Rpm-maint] New RPM seems to be much more aggressive about stripping binaries

2008-09-02 Thread Ville Skyttä
On Tuesday 02 September 2008, Richard W.M. Jones wrote: I don't know if this is the new version of RPM in Fedora Rawhide, or just coincidence, but recently I've noticed that RPM has become even more aggressive about stripping binaries. Just out of interest, how? Is strip invoked bu rpmbuild

Re: [Rpm-maint] nss/nspr location for configure

2008-11-26 Thread Ville Skyttä
On Wednesday 26 November 2008, Jan Engelhardt wrote: rpm's configure script cannot find nss/nspr even though these are installed. As it turns out, configure.ac does not use nss/nspr's pkg-config files Unless I'm mistaken, upstream nss and nspr do not ship pkgconfig files, they're distro

[Rpm-maint] [PATCH] Improve Qt support in find-lang.sh

2009-01-29 Thread Ville Skyttä
Hello, The attached patch improves Qt translation support in find-lang.sh, especially the --all-name part of it. I suppose this could be implemented in a prettier way by someone with better sed-fu than mine, but this works for me. One test case is Qt 4 (/usr/share/qt4/translations in Fedora

Re: [Rpm-maint] RPM 4.7.0 released!

2009-04-16 Thread Ville Skyttä
On Thursday 16 April 2009, Panu Matilainen wrote: More detailed compatibility information about the new features is available in http://rpm.org/wiki/Releases/4.7.0#Compatibilitynotes I would add a compatibility note about *.py in s?bin dirs no longer being byte-compiled: if one used %exclude

Re: [Rpm-maint] [PATCH 01/15] Generalize python configuration code to support both Python 2.* and Python 3.*

2009-10-16 Thread Ville Skyttä
On Thursday 15 October 2009, David Malcolm wrote: +WITH_PYTHON_INCLUDE=`${PYTHON} -c 'from distutils.sysconfig import *; import sys; sys.stdout.write(%s\n % get_python_inc())'` I think the last statement could be just sys.stdout.write(get_python_inc()), no need for the newline.

Re: [Rpm-maint] Patch: repair nosource/nopatch tag insertion

2010-04-09 Thread Ville Skyttä
On Friday 09 April 2010, Michael Schroeder wrote: Maybe NOSOURCE/NOPATCH should not be internal at all, though. They are useful to check if a rpm is src or nosrc, there is no other way to detect this. I don't know if it's correct, but in rpmlint we decide that based on whether the source rpm

Re: [Rpm-maint] Enforce install/erase order without adding a Requires:

2010-11-25 Thread Ville Skyttä
On Thursday 25 November 2010, FlorianFesti wrote: Hi! There have been various issues with packages that demand a special order of installation but do not want to Require the package to be installed first. So a tag that is like Requires: during ordering but ignored otherwise is needed.

Re: [Rpm-maint] Patch: Add support of syslog when installing/removing packages

2011-03-03 Thread Ville Skyttä
On 03/03/2011 05:36 PM, seth vidal wrote: rpm[$pid:[$processname|$depsolvername]] pkgname pkgstate/action I wouldn't make it easy to add anything but $appname[$pid] to that part of the log entry; I suspect it could cause problems with existing software that parses the logs and expects to

[Rpm-maint] lua extension API stability

2011-06-04 Thread Ville Skyttä
Hello, I'm considering using lua in some scriptlets, and noticed that while posix.symlink() currently exists in the posix extension included with rpm, it no longer exists in the current upstream version, there's a 3rd argument to posix.link() instead for that purpose:

[Rpm-maint] [PATCH] Spelling fixes.

2013-02-16 Thread Ville Skyttä
--- CHANGES| 20 INSTALL| 12 +- autodeps/freebsdelf.req| 2 +- autodeps/hpux.prov | 2 +- autodeps/osf.req | 2 +- lib/fsm.c | 4 ++-- lib/header.c

[Rpm-maint] [PATCH] Search for 7za and 7z for 7-zip.

2013-07-29 Thread Ville Skyttä
--- configure.ac | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure.ac b/configure.ac index ec85ede..7351291 100644 --- a/configure.ac +++ b/configure.ac @@ -90,7 +90,7 @@ fi dnl dnl Find some common programs dnl -AC_PATH_PROG(__7ZIP, 7zip, /usr/bin/7za, $MYPATH)

[Rpm-maint] [PATCH] python: flake8 cleanups

2014-04-05 Thread Ville Skyttä
--- python/rpm/__init__.py| 18 -- python/rpm/transaction.py | 15 ++- 2 files changed, 22 insertions(+), 11 deletions(-) diff --git a/python/rpm/__init__.py b/python/rpm/__init__.py index 196be3c..3b750db 100644 --- a/python/rpm/__init__.py +++

Re: [Rpm-maint] [PATCH] perl.{prov,req} improvments

2014-09-18 Thread Ville Skyttä
On Thu, Sep 18, 2014 at 9:38 AM, Jerome Quelin jque...@gmail.com wrote: This patch only acknowledges this fact, and allows _ in versions. This results in the underscores also ending up in the Provides. I don't think that's a good thing as IIRC rpm's version comparison does not treat it the same

Re: [Rpm-maint] [PATCH] perl.{prov,req} improvments

2014-09-18 Thread Ville Skyttä
On Thu, Sep 18, 2014 at 1:38 PM, Michael Schroeder m...@suse.de wrote: Well, 0.95_01 is actually more correct than 0.9501. Perl's versions are actually more or less floating point numbers, thus 0.9501 is supposed to be less than 0.96. From rpm's side, '_' is exactly equivalent to '.', so

[Rpm-maint] [rpm] brp-python-hardlink: Use cmp instead of sha1summing (#30)

2015-11-13 Thread Ville Skyttä
diffutils is a rpm-build dependency anyway nowadays, e.g. find-debuginfo.sh uses cmp too. See PR #16 for discussion. You can view, comment on, or merge this pull request online at: https://github.com/rpm-software-management/rpm/pull/30 -- Commit Summary -- * brp-python-hardlink: Use cmp

[Rpm-maint] [rpm] pythoneggs.py: flake8 fixes (#31)

2015-11-13 Thread Ville Skyttä
...except long lines warnings. You can view, comment on, or merge this pull request online at: https://github.com/rpm-software-management/rpm/pull/31 -- Commit Summary -- * pythoneggs.py: flake8 fixes -- File Changes -- M scripts/pythoneggs.py (40) -- Patch Links --

[Rpm-maint] [rpm-software-management/rpm] Spelling fixes (#70)

2016-06-04 Thread Ville Skyttä
You can view, comment on, or merge this pull request online at: https://github.com/rpm-software-management/rpm/pull/70 -- Commit Summary -- * Spelling fixes -- File Changes -- M doc/rpmspec.8 (2) M lib/rpmfi.c (2) M lib/rpmtriggers.c (2) M misc/rpmxprogname.c (2) M

[Rpm-maint] [rpm-software-management/rpm] Various python fixes (#71)

2016-06-04 Thread Ville Skyttä
You can view, comment on, or merge this pull request online at: https://github.com/rpm-software-management/rpm/pull/71 -- Commit Summary -- * python: Fix signalsCaught() docstring * python: Trivial code cleanups * python: Remove unnecessary shebang * python: Close file in _f2hdr also

[Rpm-maint] [rpm-software-management/rpm] find-lang.sh: Add --with-kde KF5 support (#112)

2017-01-01 Thread Ville Skyttä
You can view, comment on, or merge this pull request online at: https://github.com/rpm-software-management/rpm/pull/112 -- Commit Summary -- * find-lang.sh: Add --with-kde KF5 support -- File Changes -- M scripts/find-lang.sh (13) -- Patch Links --

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

2017-06-29 Thread Ville Skyttä
I don't see how this change would affect that use case at all. Before this change, the macros were expanded using `%{__python}`, and they still are (the shebang in the scriptlet is not used). -- You are receiving this because you are subscribed to this thread. Reply to this email directly or

[Rpm-maint] [rpm-software-management/rpm] Spelling fixes (#241)

2017-06-25 Thread Ville Skyttä
You can view, comment on, or merge this pull request online at: https://github.com/rpm-software-management/rpm/pull/241 -- Commit Summary -- * Spelling fixes -- File Changes -- M CHANGES (18) M build/files.c (4) M doc/hacking.doxy.in (2) M doc/librpm.doxy.in (2) M

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

2017-06-25 Thread Ville Skyttä
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-310887287___ Rpm-maint mailing list

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

2017-05-21 Thread Ville Skyttä
`.py` suffixes removed. I don't think there's anything fragile about this particular use of `print`, nor does it require the `print_function` import. It is also nothing new, the previous `%python_sitelib` and `%python_sitearch` definitions used it too. -- You are receiving this because you

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

2017-05-21 Thread Ville Skyttä
Oh well, anyway, since it was not my purpose to actually make the sys.stdout -> print changes, they're back to sys.stdout.write in this latest revision. -- 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] Python macro improvements (#221)

2017-05-21 Thread Ville Skyttä
Oops, the print stuff is like I said in the macros shipped by Fedora's python-rpm-macros, not here (here they were using sys.stdout.write). But the rest of what I said still stands. -- 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] Python macro improvements (#221)

2017-05-21 Thread Ville Skyttä
You can view, comment on, or merge this pull request online at: https://github.com/rpm-software-management/rpm/pull/221 -- Commit Summary -- * Use scripts instead of python -c to retrieve %python_* values * Get %python_version from platform.python_version_tuple -- File Changes -- M

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

2017-06-07 Thread Ville Skyttä
It'll take me about a week and a half until I'll get back to this. If anyone wants to ahead with this in the meantime, please feel free. -- 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] Python macro improvements (#221)

2017-06-07 Thread Ville Skyttä
`python_sitedirs` is otherwise fine, but it will also in its current form be used to retrieve the python version, which seems a bit odd to me. Also, perhaps it will be extended to generate some new other macros in the future. I'm currently thinking along the lines of `python-macrotool` or

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

2017-06-19 Thread Ville Skyttä
Combining done, script renamed to python-macro-helper. I think the installation point in rpm's dir structure serves as a prefix/namespace already. -- 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] Python macro improvements (#221)

2017-08-28 Thread Ville Skyttä
scop commented on this pull request. > @@ -0,0 +1,27 @@ +#!/usr/bin/python -tt Shouldn't really matter all that much, because the script is invoked with an explicit "outside" interpreter without the shebang affecting anything, but meh... dropped. -- You are receiving this because you are

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

2017-08-28 Thread Ville Skyttä
@scop pushed 1 commit. 9e192ce python-macro-helper: Drop -tt from shebang -- You are receiving this because you are subscribed to this thread. View it on GitHub:

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

2017-08-28 Thread Ville Skyttä
Rebased. @pmatilai 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-325380081___ Rpm-maint mailing list

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

Re: [Rpm-maint] [rpm-software-management/rpm] Invoke python-macro-helper with -Es python args (#341)

2017-10-25 Thread Ville Skyttä
> I don't think that passing -E is good Could you elaborate on that a bit, especially when you think -s /is/ good? Without -s, code from user site packages is loaded, potentially interfering with the commands. Without -E, PYTHONPATH from environment can be used to do exactly the same. (If

[Rpm-maint] [rpm-software-management/rpm] Invoke python-macro-helper with -Es python args (#341)

2017-10-25 Thread Ville Skyttä
To limit environment and user home dir influence. You can view, comment on, or merge this pull request online at: https://github.com/rpm-software-management/rpm/pull/341 -- Commit Summary -- * Invoke python-macro-helper with -Es python args -- File Changes -- M macros.in (6) M

Re: [Rpm-maint] [rpm-software-management/rpm] License of find-lang.sh is ambiguous around modification (#595)

2018-11-09 Thread Ville Skyttä
- [x] @scop ok with relicensing under MIT -- 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/595#issuecomment-437430364___

Re: [Rpm-maint] [rpm-software-management/rpm] License of python-rpm-generators (#471)

2019-04-20 Thread Ville Skyttä
LGPLv2.1(+) or GPLv2(+) is fine with me. -- 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/471#issuecomment-485118161___