Saturday, December 14, 2024

Re: devel/intellij-fsnotifier: new port

14.12.2024 17:19, Kirill A. Korinsky пишет:
> Both Intellij IDEA and Pycharm ships fsnotifier which is compiled for Linux,
> and which doesn't work on OpenBSD.
>
> Anyway, here an updated diff and tgz:

Looking good, except

- I typoed the package name in the diff
- you removed post-install completely incl. the binary rename
- config line in README has a trailing space

This is an optional filesystem notifier replacement for JetBrains IDE
such as IntelliJ IDEA, PyCharm and so on. This fsNotifier enable it via
adding the following line to
~/.config/JetBrains/<product><version>/idea.properties:

idea.filewatcher.executable.path=${TRUEPREFIX}/bin/intellij-fsnotifier

Should be plural "IDEs".

"This fsNotifier" should go or be the object, e.g. "Enable this fsNotififer
via <config snippet>"; my bad, I was probably too brief; if you want to
keep "adding ..." it shouldsay "by" not "via".

With above fixed, OK kn

"such as A, B and so on" could just be "like A or B" to brevity, but I don't
want to nit pick too much on phrasing when it's already correct - as you like.


> Index: devel/intellij/pkg/README
> ===================================================================
> RCS file: /home/cvs/ports/devel/intellij/pkg/README,v
> diff -u -p -r1.6 README
> --- devel/intellij/pkg/README 25 Jan 2023 10:54:32 -0000 1.6
> +++ devel/intellij/pkg/README 14 Dec 2024 14:17:21 -0000
> @@ -13,6 +13,10 @@ Native Libraries
> ================
> Functionality depending on native libraries is not likely to work.
>
> +Filesystem notifier
> +===================
> +Install the intellij-fsnotfier package to use fsNotifier.

I forgot an 'i' in the package name.

> +
> Errors After Upgrading
> ======================
> If you receive errors starting after an upgrade, try the following:

No comments:

Post a Comment