Louis Sautier

Results 69 comments of Louis Sautier

Hi @tartley. If your PR works, it means tests are included so my PR would not be needed :)

No worries, it's not a massive problem. We (Gentoo) simply use GitHub releases instead of PyPI packages for now. Howevern, it is good practice to include tests in sdists if...

> If such a test fails for you, but passes for me (the only scenario when it is useful to have bundled them) then that (to my way of thinking)...

Even with `-s`, tests fail if stdout isn't a TTY (easily reproduced, for instance with `pytest -s | cat`). Is there a workaround? EDIT: found one: `script -eqc "pytest -s"...

I agree, @nfnty, could you please look into merging this?

Hi, thanks for the reply. Actually my PR isn't perfect either as there is no way to build the manpage but not the rest of the doc. Let's wait until...

Actually please don't merge this yet, `--size 0-18446744073709551615.1` doesn't trigger an error.

I don't really know why that fraction is supposed to fail here: https://github.com/sahib/rmlint/blob/427791ca7c9ffa66b2c666b6b01874c5e165fae6/lib/cmdline.c#L387 I'll leave this PR in its current state, as an incomplete fix for #522.

It looks like older versions of Python should be removed from Travis.

Hey @blueyed, On Gentoo, we have [this patch](https://github.com/gentoo/gentoo/blob/9fa39e61bd93129deaf4f0e8701014cd2f95cc2e/dev-python/pypy3/files/7.3.1-gentoo-path.patch#L23) which makes distutils install `data_files` into `/usr`. I don't really know why [`syconfig.py`](https://github.com/python/cpython/blob/master/Lib/sysconfig.py) and [`distutils/command/install.py`](https://github.com/python/cpython/blob/master/Lib/distutils/command/install.py) contain different `INSTALL_SCHEMES` (links are for CPython...