Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature: Colorize filtered output #58

Conversation

alacasta
Copy link
Contributor

@alacasta alacasta commented Apr 5, 2020

What?

This contribution solves #56 .

The contribution is about parsing the output of the logging and show it based on the color selected when creating the filter.

To create a filter

  • Add a new text in the filterbox
  • Click on Set Color and a color-picker will be open with a random color (see screenshot)
  • Select your preferred color
  • Click on Filter

image

Example

The following example, contains two filters (the ID of the device has been removed):
image

@alacasta alacasta changed the base branch from master to windows-build April 5, 2020 19:59
@alacasta alacasta marked this pull request as ready for review April 5, 2020 20:03
@alacasta
Copy link
Contributor Author

alacasta commented Apr 5, 2020

@pentix
Can you please take a look to this? :)

@alacasta
Copy link
Contributor Author

alacasta commented Apr 5, 2020

(note that i've reverted your changes to make the tool working on windows, but they have not been included in this PR)

@pentix pentix added this to the v0.7 milestone Apr 5, 2020
@pentix
Copy link
Owner

pentix commented Apr 5, 2020

That looks really cool, I'll take a look at it as soon as the Windows stuff runs smoothly.
As this is a new feature it should be included in the next release, so I'm adding the v0.7 milestone to it.

@alacasta
Copy link
Contributor Author

alacasta commented Apr 6, 2020

Reopening to enforce a newer CI cycle

@alacasta
Copy link
Contributor Author

alacasta commented Apr 6, 2020

Definitively, something changed. Yesterday, it worked and no changes happen in this branch :(

Pleas, help @pentix! 🙏

@pentix
Copy link
Owner

pentix commented Nov 1, 2020

Hi Asier!

I hope you're well! Sorry for my late answers, I did not forget your pull request but found very little time during the summer to work on my projects. I just wanted to let you know that I did not forget your work here and will try to get all the build system stuff done and merged, and then have a look at this.

Thank you!

@pentix
Copy link
Owner

pentix commented Dec 25, 2020

Definitively, something changed. Yesterday, it worked and no changes happen in this branch :(

I couldn't quite figure out what changed, i.e. why it worked in the first build, however as you mentioned some of the Qt features this PR uses was introduced in Qt 5.10

Ubuntu (especially the LTS version that the CI uses) tends to lag behind,
the package search shows that 5.9.5 is the current version supplied there.

It's the same as with the libssh dependency: To fully support this, a user would need to build the required libraries himself, and also to load and install the required header files, which is fine for the small libssh, but unfeasible for the giant Qt.

An option would be to let the build system decide whether these features are available, and compile them only if possible, i.e. if version(Qt) >= 5.10; But then, would it still be possible to have this feature somehow, i.e. with another coloring? And how would we need to update the build system for this to work on all 3 platforms?

I know that's unsatisfying, however if I straight up add Qt >= 5.10 as a dependency, there will be a lot of users telling me compilation doesn't work for them. (As this dependencies are dynamically linked on Linux, just providing a .deb doesn't work, since it will fail to load the correct Qt version on execution)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants