Remove auto-detection of optimal buffer size

Since we don't pass zero anymore, this code is never called. Replace
it with 'zero means hardcoded initial value', since that hardcoded
value is what we were already passing, and change all the callers to
pass in zero (as 'auto-compute' again) instead, for simplicity.

Codecov: disable gcov

Coverage informatio is already processed by lcov, so let's use that
directly. Should behave more close to coveralls.

Name codecov uploads by the Python version

This way the artifacts in codecov are much clearer.

Remove verbose from the codecov action

Update README to point to github actions and badge

… instead of Travis.

Shorten the name of the CI

Remove the travis config

GitHub Actions seems pretty stable now, let's switch over.

Enable codecov as well

Enable coverage and coveralls upload in github actions

Initial github actions-based continous build

Preparing to migrate from Travis…

Store doc/readme and doc/news in the git repo

… instead of rebuilding them every time. This is in order to fix the
readthedocs "build", since it only builds docs (via sphinx) but
doesn't run the actual build.

Remove compatibility with old recommonmark versions

It's hard trying to keep compatibility with both very old (<1.8),
old (1.8) and modern (Sphinx), while also supporting old recommonmark;
recommonmark 0.5 was released in January 2019.

So let's drop the old support and just keep support for Sphinx 1.8 and
3.0+, and more modern recommonmark.

Release version 0.7.2

Add recommonmark to sphinx extensions

With Sphinx 3.0+, the old way of enabling markdown is completely
broken, so without this the documentation is not really working…

Run Python 3.9 tests too

Both locally and on travis.

[Fixed] typo in readme.md

`sudo install python3-pyxattr` changed to `sudo apt install python3-pyxattr`

Fix travis config after tests rename

Rename test/ → tests/

More consistency between my packages

Add unreleased changelog

Rework and expand the mixed access test

This is more of a kernel xattr test, not the library, but good
enough to guarantee mixed access from Python.

setup.py: add one more classifier topic

Tests: ensure resources as closed in subject tests

A bit of boilerplate, but gets rid of the leaking FD problem.

Bump version for 0.7.1 release…

Oops, fix bug tracker link

Bump version numbers for new release

No sense in waiting more, this is good conversion to Python 3 +
feature request + a bug fix.

Switch coverage badge to codecov

Still issues with merging reports in coveralls.

Check return value from PyList_Append

This fixes the correctness aspect of #17, although not the entire
point.

Expand setup.py configuration

The current pypi project page is somewhat bare… and shields.io can't
extract metadata from it.

Run tests on file io streams as well

Add support for Path-like objects in Python 3.6+

This is done by switching to PyUnicode_FSConverter (3.1+), which
supports it. The convert_obj() function is now much simpler, which is
a bonus.

Expand tests to check behaviour with path objects, and skip those on
Python 3.6.

Closes #20.

Start updating NEWS for upcoming release

Otherwise I forget what's in :)

Switch namespace_none test to check all calls

Instead of all input parameter types for one call. This is better
since more calls are checked, and the parameter type checking is not
the goal of this test.

Travis: cache codecov as well

Since it's anyway installed.

Run name tests as (encoded) bytes too

Tests: remove_on_missing applies to symlinks as well

Tests: small py3 cleanup

Remove support for Python 2 in the extension module

Simplifies a bit the code, but not much (yet?).

Disable MacOS builds

Apparently Python is not actually supported, so the Python 2 builds
were working by accident. Sigh sigh sigh…

Makefile: add a simpler test target

Split the other monolitic test function

Yay, cleaner now. But lots of many test invocations :)

Split the monolithic ListSetGetDeprecated test

The failure modes are better tested separately, and leave this main
test as set-get-remove simple test.

Introduce an any_subject fixture which includes symlinks

Only symlink creation fails for user attributes on symlinks, so tests
which only list/get do work. This allows removing explicit tests for
symlinks.

Add explicit test for symlink creation failure

Don't compute coverage for external files

Usually includes are caught here.

Reduce manyops count

We test more (object types), so let's reduce the call count for
each. Too slow otherwise.

Convert test suite to pytest

Much more parametrisation, which means:

- more test cases covered (testing more/all object types in most
  functions)
- more granular tests, which should help diagnose of failures

tests: stop trying to set user attributes on symlinks

According to xattr(7), by design setting/modifying user attributes on
symlinks is not allowed, due to the way access control works, so stop
trying to test it.

Stop supporting Python 2

Bump minimum Python version to 3.4, which covers e.g. in Debian even
old-old-stable (Jessie), which is good enough.

This will allow code simplification and supporting new
features (e.g. to implement #20).

tests: move helper function outside the test class

Not sure why these were there in the first place…

tests: move constants outside the class

In preparation for pytest conversion.

Travis: remove hack for Python 3.7 test

Current config was testing 3.7 twice :/

Only export COVERALLS_PARALLEL in coveralls upload

Having it shown as per-job env makes things more confusing.

Travis: test on py3.7 and 3.8 too

Also export coverage results to codecov

Will have to decide which to keep, but codecov at least supports
partial line coverage (while missing nicer overall dashboard :/).

Add a distcheck Makefile target

Too bad setuptools doesn't have it built-in…

Switch README file to Markdown and expand it

Let's get a bit more up with the times.

Fix a docstring formatting issue

This resulted in Sphinx altering the layout significantly.

Switch build system to Python 3

But still allow easy override (make ... PYTHON=python2).

Update Travis config to export coverage information

Also enable newer Python (3.7, so not so new) under Linux.

Travis: remove obsolete python versions

Some of them, at least.

Travis: remove "sudo:false"

Per the deprecation of container-based
builds (https://blog.travis-ci.com/2018-11-19-required-linux-infrastructure-migration),
remove the sudo:false setting.

Travis: add python 3.5 and 3.6 to build matrix

Tests pass locally, so all seems good.

Travis: remove python 3.2 from build list

It seems the setuptools libraries that are installed in the travis
environments don't support anymore 3.2, which causes the builds to be
wrongly marked as failed.

Further test suite fixes to symlink ops

I forgot about the deprecated ones…

Update NEWS for proper contents of 0.6.1 release

Travis build: remove dependency on libattr-dev

One more cleanup as part of the migration to the glibc header.

Fix nofollow not work in MAC

Tests: fix symlink testing

On Linux, non-root users are not allowed to set extended attributes,
so normal testing has hidden this. A fix to the Mac OS support
proposed in #14 uncovered this problem, which led to investigation and
this fix.

Manual testing on Linux with root and namespace=trusted now passes the
tests.

Tests: introduce an alias for NS_USER

Using NS_USER directly makes it hard to change it for a one-off manual
test, so let's virtualise it, and additionally allow overriding from
the command line.

Release GIL when do I/O operations

Release GIL in _list_obj, _get_obj, _set_obj, and _remove_obj. These
functions do I/O operations which take long time. We should release
GIL to allow other threads work.

Further improvements to benchmark code

Makefile: move list of python versions to a variable

This way, it can be overridden from the command line, to allow a more
targeted test/benchmark.

Add a mini-benchmark suite

This will make it easy to see the impact of changes, for example the
one proposed in #16.

Bump version to 0.6.1

Also switch downloads repository to HTTPS.

Remove use of and mentions of the attr library

Commit a799657 removed its use, so let's drop the mention and the
linking against it.

xattr.c: There is no more attr/xattr.h with >=attr-2.4.48

See also:
http://git.savannah.nongnu.org/cgit/attr.git/commit/?id=7921157890d07858d092f4003ca4c6bae9fd2c38

Version bump for releasing 0.6.0

Signed-off-by: Iustin Pop <iustin@k1024.org>

Expand the testing of invalid arguments

This is not significant advantage, but it does increase coverage. And
any coverage (even if not testing the logic too much) is better than
no coverage.

Test writing large values

This exercises one more code path.

Also run tests using python-dbg

Add a makefile coverage target

And expand a tiny bit the test suite to exercise one more error
path. Current coverage is not bad, ~87% lines coverage. For some
reason, no branch coverage is available :/

Rework getting and listing attributes

Currently, getting and listing attributes is inconsistent in how it
deals with allocating memory for the data returned by the syscall:

- in most cases, we ask the kernel for the value, allocate memory,
  then retrieve the value; if the value changed (increased) in the
  meantime, this will lead to what should be preventable failures
- in the single case of getall, there is a loop for getting the
  individual values, but not for initially get the list of values

Hence the bug report #12. This rather large change refactors a lot of
the code:

- doing a get/list is abstracted away into a helper function that
  always does retries until we can read the value or fail with a
  different error than ERANGE; this helper deal with all allocations
  and resize operations.
- this means most of the single get/list operations are heavily
  simplified
- and also means that we can always start optimistically with an
  initial buffer size (currently set to 1K), instead of asking the
  kernel first: this saves one out of two syscalls in the case the
  value will indeed be smaller, but adds one extra syscall (the failed
  initial read) in the case it's not.

The optimisation is a double-edged sword: for small attributes, it
will be a win (e.g. the test suite is ~5% faster now), but for large
attribute/lists, it will be potentially slower (50% more
syscalls). Not sure how to nicely deal with this; it would be good to
have a keyword argument maybe? Or build flag? Left for future
enhancements.

Closes #12.

A few small improvements to the test suite

Especially testing empty attribute value and ensuring not found
attribute raises an exception.

Initialise target_t members on error paths

This is a no-op from the point of view of the actual code flow, but
not doing so can trigger false positives in some versions of GCC (4.9
from Jessie) at high optimisation levels (-O3). For an interesting
discussion of why this happens at `-O3` vs. at `-O2`, see
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60165.

Closes #13.

Spelling fix in documentation

Fix name space information in documentation

Closes #11.

Revert "Add build status badge to README"

This reverts commit de9bc00d145874b5a7a63ef382de215b735e8961. First,
it was markdown and README is rst, but that's not the point: sphinx
wants to warn about 'non-local image in URI', and the silencing of
this is only present in 1.4 and later, so for now just revert.

Add build status badge to README

Mark OSX build non-experimental

Also override Python list, since we only care about system python (OSX
doesn't support yet multiple Python versions), and the current state
shows a strange build summary (one build with N python versions).

Fix tabs in travis config

Try to enable OSX builds in travis configuration

Based on recent patch, the test suite should pass, so let's try to
enable it.

Merge pull request #9 from ahknight/osx-support

Added support for OS X's slightly different xattr calls.

Added support for OS X's slightly different xattr calls.

Bump version to 0.5.6

Pending commits fix compilation issues due to sign-compare, let's make
a very much overdue release.

Fix building on systems which don't have ENODATA

This patch (a modified form of the original patch from Cyril Brulebois
<kibi@debian.org>) fixes building on systems which don't have ENODATA
(e.g.  GNU/kFreeBSD).

Fix potential name truncation in merge_ns()

It seems I misread the snprintf man page: a return value of exactly
the buffer size means truncation has occurred, so we need to fix the
operator (`>` → `≥`).

Additionally, improve slightly the error message raised in such a
case; this shouldn't ever happen.

travis-ci: less build log noise

Surprisingly, the integration works! But we can reduce some noise in
the build log:

- be explicit that we don't require sudo (silences YELLOW! notice)
- override install since we don't require/use pip requirements

Always enable -Wsign-compare

The code is clean, and this is an important warning (due to extensive
use of both signed and unsigned integers), so let's enable it.

Fix sign-compare warning in merge_ns()

Thanks to travis-ci - the code does what seems an unsafe sign compare,
but actually it is a valid comparison (we test/handle the `< 0` case
as well, which is the only way the comparison could be invalid). To
silence this, do an explicit (safe) cast.

Add a .travis.yml file

Not sure if the travis-ci environment supports C extension modules or
attributes on filesystems, but let's try.