| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
| |
| |
| |
| |
| |
| |
| | |
D-Feet's test unit does not specify the required introspection
version.
This adds the required introspection version.
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The test unit contains hard coded values for paths. However, these
hard coded variables consider that the source code path and the
build path are the same, something usual in autotools.
These hard coded values has been replaced by build time variables
that take the appropiate values considering different build paths.
|
| | |
|
| | |
|
|/ |
|
| |
|
| |
|
| |
|
| |
|
| |
|
|\
| |
| |
| |
| | |
Follow GNOME 3.32 menu guidelines
See merge request GNOME/d-feet!13
|
| |
| |
| |
| | |
to match the .desktop file and the updated about menu item label
|
|/
|
|
| |
https://gitlab.gnome.org/GNOME/Initiatives/wikis/App-Menu-Retirement
|
| |
|
| |
|
| |
|
| |
|
|\
| |
| |
| |
| | |
Correctly honour theme foreground colour
See merge request GNOME/d-feet!12
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Rather than try to explicitly look up the colour, don't specify a colour
at all. Argument names are never nested within a <span> applying a
different foreground colour.
This does have one visible effect: with Adwaita (light), selecting the
treeview row causes method argument names to switch from black to white,
just like the method name. I think this is actually more readable (and
the green used for the signature is illegible against the blue
selected-row highlight.)
As discussed in https://gitlab.gnome.org/GNOME/d-feet/merge_requests/4#note_319094 and !8.
|
| | |
|
|\ \
| | |
| | |
| | |
| | | |
Handle Unix FD type when pretty-printing types
See merge request GNOME/d-feet!11
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This was added to D-Bus almost a decade ago. Without this change it
shows up as 'Error(h)' in method signatures.
(This doesn't mean we support passing them as in-parameters.)
|
| | | |
|
| |/
| |
| |
| |
| | |
It's legal to slice past the end of the string; you get back an empty
string in that case.
|
| | |
|
| | |
|
| | |
|
|/ |
|
| |
|
|\
| |
| |
| |
| | |
Cleanup doap file
See merge request GNOME/d-feet!10
|
|/
|
|
|
|
|
| |
- Fix bug link and use the new gitlab issues page
- Drop John from the maintainers list. He's not working on the project
any longer. Thanks for all the work!
- Mention the used programming language
|
|\
| |
| |
| |
| | |
Fix bugs introduced in MR 7
See merge request GNOME/d-feet!9
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
intltool-update runs during make check and finds files inside app:
The following files contain translations and are currently not in use. Please
consider adding these to the POTFILES.in file, located in the po/ directory.
app/files/share/d-feet/ui/addconnectiondialog.ui
app/files/share/d-feet/ui/app-menu.ui
app/files/share/d-feet/ui/executedialog.ui
app/files/share/d-feet/ui/introspection.ui
app/files/share/d-feet/ui/mainwindow.ui
|
| |
| |
| |
| |
| |
| | |
This would have caught my bug where the CI-built Flatpak bundle still
tried to use Python 2 despite it only being in the GNOME SDK, not the
GNOME Platform.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This will allow us to run `make check` during CI flatpak builds, which
manually build the d-feet module rather than building it with
flatpak-builder. We still use --disable-tests in the real manifest
because the tests require an X (or Wayland) server.
Specifying "cleanup": ["*"] causes all files installed by the
pycodestyle module to be removed at the end of the build process.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
For a "normal" build running flatpak-builder to completion, the
'build-options: env: PYTHON: /usr/bin/python3' specified at the top
level of org.gnome.dfeet.json takes effect. However, the .gitlab-ci.yml
(lifted from other projects) uses --stop-at to build everything except
d-feet with flatpak-builder, then runs the steps to build d-feet by
hand. The PYTHON environment variable does not persist from the earlier
steps.
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Trying to compile this regexp fails with:
re.error: unbalanced parenthesis at position 5
I introduced this bug in 45a1146e1122b376ddac312687e8e27d4e0a8076. Fix
it, and add a small test case that would have caught it.
|
|/ |
|
|\
| |
| |
| |
| | |
Fix 'make check', detect 'pycodestyle' (the new name for 'pep8'), use Python 3 in Flatpak version, and set up CI
See merge request GNOME/d-feet!7
|
| |
| |
| |
| |
| | |
The Flatpak part is cargo-culted from Glade and other projects of its
ilk; the Fedora part is derived from gnome-initial-setup.
|
| |
| |
| |
| |
| | |
Python 2 is not included in the 3.30/master runtime (though it is
included in the SDK).
|
| |
| |
| |
| |
| |
| |
| | |
../../src/dfeet/settings.py:22:17: W605 invalid escape sequence '\.'
../../src/dfeet/settings.py:23:6: W605 invalid escape sequence '\s'
../../src/dfeet/settings.py:28:10: W605 invalid escape sequence '\s'
../../src/dfeet/settings.py:29:12: W605 invalid escape sequence '\('
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The following code:
self.connection = Gio.DBusConnection.new_for_address_sync(
self.__bus_address,
Gio.DBusConnectionFlags.AUTHENTICATION_CLIENT |
Gio.DBusConnectionFlags.MESSAGE_BUS_CONNECTION,
None, None)
causes newer versions of pycodestyle to complain:
../../src/dfeet/bus_watch.py:139:17: W504 line break after binary operator
However, if I move the the line break to be before the binary operator:
self.connection = Gio.DBusConnection.new_for_address_sync(
self.__bus_address,
Gio.DBusConnectionFlags.AUTHENTICATION_CLIENT
| Gio.DBusConnectionFlags.MESSAGE_BUS_CONNECTION,
None, None)
Older versions complain in the opposite way:
../../src/dfeet/bus_watch.py:140:17: W503 line break before binary operator
The thread around
https://mail.python.org/pipermail/python-ideas/2016-April/039774.html
explains why the tool has changed its mind.
I prefer the old style, so let's just ignore the new warning.
|
| |
| |
| |
| | |
These were all reported by flake8.
|
| |
| |
| |
| | |
These are considered Bad Style, and pycodestyle reports them as such.
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The 'pep8' tool has been renamed upstream to 'pycodestyle'. On Debian,
the Python 3 version is packaged and installed as pycodestyle. On
Fedora, it's packaged as python3-pycodestyle and installed as
pycodestyle-3; the Python 2 version is installed as pycodestyle-2 with a
symlink from pycodestyle. Make configure search a variety of such names,
and use the path it finds.
|
| |
| |
| |
| |
| |
| |
| | |
Since c28acbf326bb79543b285f371bdf068d8831d9a7 the application has used
the Python version detected at configure time, but tests were still
unconditionally run with the system version of Python 2. This made it
impossible to run the tests on systems with only Python 3.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
These were introduced in e6488e6c0fd0416c54ecff1dd682f0ae8aa4c6fb and
e6488e6c0fd0416c54ecff1dd682f0ae8aa4c6fb:
src/dfeet/introspection_helper.py:9:1: E302 expected 2 blank lines, found 1
src/dfeet/introspection_helper.py:14:100: E501 line too long (107 > 99 characters)
src/dfeet/introspection_helper.py:17:1: E302 expected 2 blank lines, found 1
src/dfeet/introspection_helper.py:20:1: E302 expected 2 blank lines, found 1
src/dfeet/introspection_helper.py:23:1: E302 expected 2 blank lines, found 1
|
|\ \
| |/
|/|
| |
| | |
Make fields in Execute Method dialog selectable
See merge request GNOME/d-feet!6
|
| |
| |
| |
| |
| |
| | |
This dialog is not particularly short of space, and it's nice to have
the full (bus name, object path, interface, method name) quad visible in
one place.
|