Julian Gilbey

Results 106 comments of Julian Gilbey

Oh dear. I removed the `pytest-timeout` plugin and that made things differently bad: ``` test/test_plugin.py:95 /tmp/autopkgtest.jdEbbc/build.5FP/src/test/test_plugin.py:95: PytestUnknownMarkWarning: Unknown pytest.mark.timeout - is this a typo? You can register custom marks to...

FYI: I've just been looking at https://github.com/fabioz/PyDev.Debugger, and see that they have a vendored version of `bytecode` with this somewhat addressed: see https://github.com/fabioz/PyDev.Debugger/commit/bcb8a28669e9178f96f5d71af7259e0674acc47c (though I confess that I do not...

OK, I think I've figured this one out: because my chroot is mounted as an overlay filesystem, `/tmp/s2t9qns8us6` is on the original device but `/tmp/s2t9qns8us6/test.txt`, which is in the overlay,...

BTW, this patch fixes the issue and shouldn't change the behaviour of the function in "normal" situations, so it may be worth applying.

No, I spoke too soon; the test: ```python topdir = find_mount_point(path_b) trash_dev = get_dev(topdir) if trash_dev != path_dev: raise OSError("Couldn't find mount point for %s" % path) ``` causes an...

Indeed! `os.expandpath(b"~")` expands to the value of `HOME` if it is set (https://docs.python.org/3/library/os.path.html#os.path.expanduser). In my (admittedly weird) use case, though, `HOME` was being inherited by the `schroot` call, but the...

I guess the only other way of resolving this is to make the modification to `find_mount_point()` that I suggested above and then remove the check on line 211: `if trash_dev...

> I guess the only other way of resolving this is to make the modification to `find_mount_point()` that I suggested above and then remove the check on line 211: `if...

I guess it depends on what you agreed with Virgil. Some projects say things like "Copyright 2012 A. N. Author, Copyright 2013-2017 A. N. Other, Copyright 2017-2022 New Author". Anyhow,...

Actually, there is a possibility: rename the Python module to `typed_argument_parser` instead of `tap`. Since it's only used in the import section, the longer name will make almost no difference....