jue89 / node-systemd-journald

Native bindings to journald
MIT License
28 stars 10 forks source link

Is it possible to merge this into `systemd/node-systemd`? #10

Closed paluh closed 6 years ago

paluh commented 6 years ago

As a developer I'm really confused because there are "official" bindings for node provided on github.com/systemd account and there is your lib which works on newer node versions (as far as I understand your README.Why section).

Have you considered proposing your bindings as a replacement for github.com/systemd/node-systemd? I mean something like opening pull request or issue there.

jue89 commented 6 years ago

Hi paluh!

I had several reasons that kept me from merging my code into the official project in the first place:

But maybe we can move this project to the systemd organisation if the systemd guys want this project inside (and give me the right to maintain it). I need some time to think about this.

I hope this helps to understand why.

paluh commented 6 years ago

Thanks for explanation, @jue89!

I understand your motivation better now. The only thing is that I'm afraid that many developers can be somewhat skeptical about future/maintenance of "unofficial" library. Ironically it is other way around in this case as you described above ;-)

Anyway, thanks for sharing your work and thoughts!

P.S. I've just published Purescript bindings to your library:

https://pursuit.purescript.org/packages/purescript-systemd-journald/0.1.1

jue89 commented 6 years ago

Nice!

And thank you for testing out the fields option. Nice collection to demonstrate what's possible.