Small simplification in ACL_init

Add a test for __setstate__ arguments

Make the code Py_ssize_t clean

Thanks python 3.8 for the hint!

Change setstate to only take bytes

This is the recommended way (well, via Py_buffer, but I don't need
that).

Move a few tests from explicit loop to parameters

Failure messages should be more better in failure case.

Fix from_acl tests for non-Linux platforms

FreeBSD doesn't have acl_cmp, so comparison via rich compare is not
defined, thus all comparisons are False. Fix tests so the equality
check is only done on Linux, and add a poor man's test via string
representation equality.

Change entry qualifier set/get behaviour

This was intended to address #13, but investigation found out more
breakage than just that. It's hard to make overflow/underflow tests
without assuming the signedness of the uid_t/gid_t types, so
assume/require that they're unsigned (it is true with glibc, MacOS and
FreeBSD) and use this to improve the behaviour:

- Fix setting very large qualifiers, both in the sense of correctly
  reporting overflow when too large, and not longer falsely reporting
  overflow for larger than signed max but smaller than unsigned max;
- Fix returning very large (larger than signed max value) qualifiers;

Fixes #13.

Minor change on how error handling style in ACL_init

Remove doc/news.rst from git

Forgot to do this when changed readme. It is now symlinked at build
time.

Convert ACL initialisation to allow file-path objects as well

This is easier than in has_extended, as we can directly use the
converter. Yay!

Convert the README to markdown

Yay, 2019.

Add stub type hints - even if not working

Apparently PEP561 only addresses packages, not modules, and somewhat
dismissively says: "code should be refactored into a package-based
distribution and indicate that the package supports typing as
described above".

Well, locally I can use it, so let's add the stub. Will see later if
it can be used somehow.

Fix tests path in MANIFEST.in

Add type: ignore to tests where we force bad behaviour

Add trove classifiers and project urls

Another xfail test

Simplify ACL.append to not duplicate Entry creation

Similar to the pattern for Entry.permset, this removes the duplication
with the (child object) init.

Add xfail test for equiv mode failures

Sigh, another bug in upstream library.

Raise better error message on un-owned but valid Entry deletion

Simplify returning None

Switch ACL to be always-initialised

This is the last object to change, but the semantics here are more
complex. Since the ACL doesn't have a parent, and the init signature
is complex, we can't detect "same-reinit", we allow arbitrary-reinit,
but this makes existing live entries be undefined; they might point to
a different entry in the new ACL, or not be valid, etc.

It could be possible to prevent re-init, but doing so requires
trickery which might be broken by serialisation, so let's just leave
it there and document it as such.

Apply no-unitialised-objects to Permset as well

This had no checks at all (Entry has one check) on attribute access.

Stop duplicating permset initialisation

A permset can be initalised either via `__init__ ` or by getting
`parent.permset` getter. The latter duplicates the logic in the
former, which is not good as hacks into the internals of the permset.

Remove the duplication by just calling explicitly permset(self) and
returning the value of it.

Expand acl_check testing for actual value checks

This actually tests upstream, but just "assert a.check()/not
a.check()" seems too little.

Restore setstate/getstate support

Sigh, another brown-bag issue. One small typo in a place, plus a
missing macro, and this went unseen for ages.

Also add a basic test for it. Probably should add Pickle as well.

Slightly expand the NEWS file

Change Entry initialisation protocol

This fixes very large and significant bugs - segfaults and memory
leaks - that were present for uninitialised object, more precisely
created but not init'ed ones.

I spent quite a bit of time thinking back on forth how to fix this,
and from the two options of:

- check initialised status on all code paths, or
- don't ever allow invalid/un-initialised objects

The latter one seems the correct one, even though the Python C API
docs imply that doing actual stuff in `__new__` should be "rare".

Tests for reference leaks and wrong re-init added as well; these would
have caught at least memory leaks before.

Standardise on capitalised error messages

For good or worse, but at least standard.

Unify attribute deletion checks for Entry

Reduce verbosity in fast-tests

Otherwise compilation issues are missed.

Cleanup py2-compatibility 'except' syntax

Remove tests/__init__.py as not needed with pytest

Include filename in exceptions raised in delete_default

And add test for failure.

Add tests for a few IO failures

Marginal test coverage increase, but checks that we do raise IOError
as expected.

Add pathlib support in apply_to and has_extended

This is done by switching these functions to PyUnicode_FSConverter,
thus removing a chunk of custom, and probably buggy, code. Easier to
read as well…

Add more test infrastructure for parameterisation

Not sure how much will be used yet…

Py3 cleanup for the module itself

Add no external to coverage target

tests: Py3 cleanup

A bit more tests

Convert TestModification to pytest

And that was the last one…

One more makefile test target

Stop testing with pypy2

Broken already :)

Standardise skip-on-pypy decorator

Convert AclExtensions to pytest

Convert WriteTests to pytest

Switch LoadTests to pytest

tests: add helper functions/fixtures

Switch travis to pytest as well

Switch to running tests via pytest

This doesn't change the test code, just the runner (for now).

Stop supporting Python 2

This just changes the documentation and stop testing with Python
2. Future patches will actually drop compatibility.

Add a fast-test target

Rename test/ → tests/

More consistency between my packages.

Run local tests with 3.8 as well

Add a distcheck target

Travis: test python 3.7 and 3.8 in standard way

This removes the hack for python 3.7, and add 3.8 as well.

Export coverage results to codecov as well

And also move the env var to just the coveralls invocation.

Badge-ify the README

Hey, it's fun!

Update Travis config to export coverage information

Bump version numbers for new release

Only 4+ years…

Note that ACL(text="...") needs a valid ACL to be passed

Closes #9.

Ensure documentation is rebuilt on source (.c) changes

Update FreeBSD notes

This mentions needing to enable ACLs on the file-system, and adds the
packages that need to be installed.

Closes #12.

Update README about build requirements

Makefile: stop trying to test with py<2.7

Since actual compat was removed a while back.

Switch documentation building to Python 3

This removes build-time dependency on Python 2, which is going away.

Travis: remove obsolete python version

Some of them, at least.

Makefile: re-enable pypy and add pypy3

Everything works again now, so let's enable them.

ACL creation: change how mode arguments are used

Currently, initialising from a mode is done by looking at the keywords
arguments that were passed and seeing if mode was part of them. For
some reason, this causes a segfault under PyPy (my fault, probably),
so let's switch to check based on whether the mode argument was
initialised. To do so, switch the type of mode to int and the Python
format char, since the 'H' is defined as "without overflow checking"
anyway, so we don't get any real checks. Document that invalid values
will cause errors.

Remove use of PyString/PyUnicode *AndDel() function

These are not documented in Python 3, so let's switch to plain
Concat() for it with explicit delete of original strings. This makes
the code buildable with pypy3.

Travis: really test 3.7

This version seems to work. Sigh…

Travis: more reworking of Py3.7 support…

Travis: rework build matrix

Apparently python 3.7 is not available on the default Trusty
distribution, so let's add Xenial for the newer Python versions. More
builds!

Drop Python 2.6 and earlier support

While the C extension doesn't need anything to support it, simply
running the tests becomes more and more difficult with such old
versions. Given upcoming Python 2 deprecation, let's drop support for
older versions and simplify the test suite.

Disable pypy tests in Makefile

With my current pypy, ACL_init triggers a SEGFAULT in pypy. Until I
can debug what's up, let's remove this test.

Increase test coverage

These are old changes from October 2016 which I forgot to commit. I
think they're related to test coverage, although a few changes (object
structure) seems a bit different.

Well, anyway, here they are.

Travis: add python 3.5/3.6 to build matrix

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.

Add a makefile coverage target

… which says that current test coverage is pretty bad - 55% line-based
and only 67% functions :(

Also run tests with debug interpreters

And bump up Python 3.x versions.

Add travis config file

This can be done now that travis environment has whitelisted the
libacl1-dev package.

Invert get_tag_qualifier() return value meaning

Returning negative value on error (rather than zero) makes the
function more friendly for cpychecker, and allows the code to be
cpychecker clean.

Add cpychecker macro for exception negative results

Per http://gcc-python-plugin.readthedocs.org/en/latest/cpychecker.html#errors-in-exception-handling.

Add cpychecker annotations for type/object correspondence

Per http://gcc-python-plugin.readthedocs.org/en/latest/cpychecker.html#associating-pytypeobject-instances-with-compile-time-types.

Bump copyright years

Rename README→README.rst

This should give better formatting, e.g. in github project page.

Add the release date to NEWS files entries

Bump version for 0.5.3 release

Update my email address

Small style change in setup.py

Merge remote-tracking branch 'github/master'

Merge pull request made on GitHub.

Cleanup all .so files on 'make clean'

Otherwise 'make test' can result in very weird error messages, if
hitting https://bugs.python.org/issue7559 (don't ask…).

tests: ignore IOErrors during operations invalid ACLs

It seems that FreeBSD at least returns EINVAL from calc_mask() and
other calls if the ACL is invalid, so let's ignore such IOErrors.

This should hopefully fix #6 and #7.

Merge pull request #8 from yaneurabeya/master

Handle FreeBSD versions greater than or equal to 7.x

tests: don't call acl_to_text on an invalid ACL

While Linux is happy to convert it to text, it seems that under
FreeBSD this doesn't (always? sometimes?) work, so let's use a proper
ACL in the str() test.

Closes #5.

Handle FreeBSD versions greater than or equal to 7.x

Sponsored by: EMC / Isilon Storage Division

Simplify a bit the module initialization

A few constants are more nicely defined.

Try to fix uid_t/gid_t usage in entry qualifiers

The current code is very broken with regards to the casting between
Python's integer type (either int in Py2 or the magic int/long in Py3)
and the uid_t/gid_t POSIX types. This means that the code is broken
outside "small" integer values.

By using uid_t/gid_t as appropriate, we can fix most of the problem
(at least as far as the new unittests are concerned). There's still no
automatic printf format modifier for them, so the code hardcodes
unsigned (which is what Linux/glibc defines them to), and also there's
a unittest that expects negative values to fail when set.

This should fix issue #3 (github).

First step towards fixing qualifier overflow

This is still a work-in-progress, since it only deals with
Python-level overflows, but it's a step forward.

Fix error message in Entry_set_qualifier

This closes #1 (github issue).