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

Project abandoned? #730

Open
bj00rn opened this issue Jun 22, 2023 · 37 comments
Open

Project abandoned? #730

bj00rn opened this issue Jun 22, 2023 · 37 comments

Comments

@bj00rn
Copy link
Collaborator

bj00rn commented Jun 22, 2023

Thanks for this awesome app!

Is this project abandoned?

A lot of open issues and PRs.

@ilgrank
Copy link

ilgrank commented Jun 22, 2023

Yes, the author is not updating the code since years and basically is also not very active here, his last activity has been a reply on this issue: #683 (comment) on Dec 19, 2022.
Lots of people asking for his return on the project or a change to the licensing that would allow others to continue developing it, but alas, he's not answering anymore.

@bj00rn
Copy link
Collaborator Author

bj00rn commented Jun 28, 2023

@ilgrank thanks, sad to hear..

@thomasnordquist would you consider loosening the license for this awesome app, so people can continue maintaining it?

@backupandforkacc
Copy link

Anyone with a good alternative to the software?

@XtremeOwnageDotCom
Copy link

Anyone with a good alternative to the software?

Despite being unmaintained for a few years at this point, It's still arguably one of the best solutions out there.

@bj00rn
Copy link
Collaborator Author

bj00rn commented Aug 28, 2023

@ilgrank thanks, sad to hear..

@thomasnordquist would you consider loosening the license for this awesome app, so people can continue maintaining it?

PING @thomasnordquist, sorry for being a pain in the ***. Just can't wrap my head around this amazing piece of tech being unmaintained!

@jeeftor
Copy link

jeeftor commented Sep 7, 2023

I suspect at some point somebody will just fork it and move it forward...

@adamalfath
Copy link

Anyone with a good alternative to the software?

MQTTX is quite good

@jeeftor
Copy link

jeeftor commented Sep 27, 2023

I still prefer this one unfortunately

@ilgrank
Copy link

ilgrank commented Feb 1, 2024

@jeeftor : no update from @thomasnordquist or @snordquist , right?

@jeeftor
Copy link

jeeftor commented Feb 1, 2024

Of course no update

@jeeftor
Copy link

jeeftor commented Feb 1, 2024

Somebody just needs to fork this and the community will likely jump in to help

@XtremeOwnageDotCom
Copy link

Somebody just needs to fork this and the community will likely jump in to help

The issue, is the license.

https://github.com/thomasnordquist/MQTT-Explorer?tab=License-1-ov-file#readme

https://www.tldrlegal.com/license/creative-commons-attribution-noderivatives-4-0-international-cc-by-nd-4-0

Long story short, the license prevents others from modifying, and then sharing this code/project.

@XtremeOwnageDotCom
Copy link

So, unless @thomasnordquist changes to a less restrictive license, or decides to start maintaining this project again-

It is as good as dead.

@jeeftor
Copy link

jeeftor commented Feb 1, 2024

Doesn't the license only come into play if it's enforced? So if it's abandoned who is going to enforce it

@bj00rn
Copy link
Collaborator Author

bj00rn commented Feb 2, 2024

Doesn't the license only come into play if it's enforced? So if it's abandoned who is going to enforce it

That's not how licenses works though.. The Non-derivative license prevents us from forking and sharing the fork.

Beside the potentially legal exposure (however unlikely) putting effort into a project without a proper license is a no-go.

@jeeftor
Copy link

jeeftor commented Feb 2, 2024

So what if the maintainer died? (Hypothetically)

@Jackfritt
Copy link

I think his family took it over to this one...
https://mqtt-explorer.com/
https://github.com/thomasnordquist/MQTT-Explorer

@MartinRiddar
Copy link

@jeeftor He isn't dead, he's just not very active on github anymore, he did a single pull request in the last year (November).
@Jackfritt If you check the Github link for that project it links here...

@scubanarc
Copy link

Doesn't the license only come into play if it's enforced? So if it's abandoned who is going to enforce it

That's not how licenses works though.. The Non-derivative license prevents us from forking and sharing the fork.

Beside the potentially legal exposure (however unlikely) putting effort into a project without a proper license is a no-go.

If you rewind the clock 5 years, it was AGPL:

https://github.com/thomasnordquist/MQTT-Explorer/tree/ea7994dad945dba2291d097ea233def9942761e3?tab=readme-ov-file

@XtremeOwnageDotCom
Copy link

If you rewind the clock 5 years, it was AGPL:

https://github.com/thomasnordquist/MQTT-Explorer/tree/ea7994dad945dba2291d097ea233def9942761e3?tab=readme-ov-file

Doesn't look like there are that many differences in the last 5 years. Perhaps we should fork that to a new project.

@scubanarc
Copy link

Yes, someone should fork it from 5 years ago when it was still AGPL, everyone move their PR's, and then we all continue on forward with this project.

@steveja
Copy link

steveja commented Mar 3, 2024

Maybe it's me - but I can't build from the git repo. 'yarn' complains that the older packages are needed (my interp'). So I think this is totally dead as an open project.

@jeeftor
Copy link

jeeftor commented Mar 3, 2024

So the license got added in this commit (2019).

9320fa2

  • Does that mean anything prior to this commit is fair game?

@XtremeOwnageDotCom
Copy link

So the license got added in this commit (2019).

9320fa2

  • Does that mean anything prior to this commit is fair game?

Basically, up to the limitations of its prior license

@thomasnordquist
Copy link
Owner

thomasnordquist commented Mar 4, 2024

Hello there,
for me time is a very limited resource, and at the end of the day I try to finally close my laptop.
This is one reason I do not maintain this project (or any other private projects) at the moment.

The license will stay like this for now, the main goal is to protect this project to be commercially distributed in any way.

My previous experiences with collborators was sadly not optimal, therefore I was discouraged to pursue this further.
I will request @bj00rn to be a maintainer of this project (since he already spend some effort to get in contact with me).

I will also help with the distribution of new versions.

There are a few things which are kind of urgent for this project:

  • A new version of electron (outdate SSL certificates)
  • Removal of the telemetry (not used anymore / opt-in telemetry)
    • since it is no longer operational removing it should be easy to do
    • option: self-hosted telemetry / vs. telemetry service free for open source usage

@scubanarc
Copy link

Hey Thomas, glad to hear you're still around! Thanks again for this awesome program. It really is the best MQTT frontend out there and I use it daily. Amazing work!

The license will stay like this for now, the main goal is to protect this project to be commercially distributed in any way.

That's very unfortunate. Please reconsider this.

It was AGPL once before, so it's already out there to be abused by companies. Keeping it CC-BY-ND 4.0 just hurts the community by preventing an active user from forking and improving it.

As of right now, it's been dead for 2 years. There are 64 PRs and no-one to merge them.

I do hope that @bj00rn is able to maintain it, but please consider opening up the license.

@GeekyTim
Copy link

GeekyTim commented Mar 4, 2024

@thomasnordquist, I'm sorry to hear you have been burnt by previous collaborations, and understand your stance. Please remember that many of us here in this community don't always have the best communication methods and often cannot see other's reasons and point of view. Being recently diagnosed as Autistic myself, I do know I have not always been the best communicator in the past.

I will not comment on your choice of license etc. That is your choice alone.

@bj00rn, I welcome any work you do on this tool. I am looking forward in particular to see it working on the Raspberry Pi, my current main machine!

@jeeftor
Copy link

jeeftor commented Mar 4, 2024

Is there a license which prohibits commercial use but opens up for open source - forking etc? Or does such a thing not exist?

@XtremeOwnageDotCom
Copy link

XtremeOwnageDotCom commented Mar 4, 2024

Is there a license which prohibits commercial use but opens up for open source - forking etc? Or does such a thing not exist?

Creative Commons NonCommercial

https://creativecommons.org/licenses/by-nc/4.0/deed.en

@bj00rn
Copy link
Collaborator Author

bj00rn commented Mar 6, 2024

@thomasnordquist thanks for the invite! I wasn't really expecting to be assigned maintainer but merely hoping to re-open a discussion on how to move this awesome project forward. That being said, I am willing to invest some time and effort to get this project up to date and try to get some momentum going here.

Let's get the janitorial side of things up to date first so we can do a new release.

  • Dependencies are way outdated
  • CI is failing.

Additionally there are a few long due, low hanging PRs that seem like they could be merged without any major considerations.

Disclaimer
Im pretty well versed in react+redux but my skills concerning electron are basic at the very best, so please be kind/patient here.

If some of you want to contribute feel free to open PRs. This will have to be a team effort going forward.

@micw
Copy link

micw commented Mar 7, 2024

Great to see that the project is not abandoned!

I was able to fix the certificate issue locally by upgrading electron to the latest release (29.0.1). I also updated electron-builder to 24.12.0, no idea if that matters. To build with a recent node version, I also added export NODE_OPTIONS=--openssl-legacy-provider && in front of the commands in the "build" script definition.

Important was to start from git tag 355e917 which was the tag the latest beta-release was made from. The changes after caused several issues. So maybe branching from there would make sense for a hotfix release.

@bj00rn
Copy link
Collaborator Author

bj00rn commented Mar 8, 2024

Great to see that the project is not abandoned!

I was able to fix the certificate issue locally by upgrading electron to the latest release (29.0.1). I also updated electron-builder to 24.12.0, no idea if that matters. To build with a recent node version, I also added export NODE_OPTIONS=--openssl-legacy-provider && in front of the commands in the "build" script definition.

Important was to start from git tag 355e917 which was the tag the latest beta-release was made from. The changes after caused several issues. So maybe branching from there would make sense for a hotfix release.

What errors du you get when upgrading electron deps on master?

@bj00rn
Copy link
Collaborator Author

bj00rn commented Mar 10, 2024

Please have a look at #757

@nicklasb
Copy link

nicklasb commented Apr 2, 2024

@thomasnordquist , @bj00rn :
Let me just say that is just GREAT!
And yeah, collaboration can be hard, especially in open source, sometimes the giving and the taking doesn't add up at all.

@bj00rn
Copy link
Collaborator Author

bj00rn commented Apr 26, 2024

@thomasnordquist Can you please have a look at master, It should be ready for a pre-release.

@thomasnordquist
Copy link
Owner

Thank you, I will work on it this week.

@bj00rn
Copy link
Collaborator Author

bj00rn commented May 2, 2024

Thank you, I will work on it this week.

Give me a shout if something is broken.

Some notes:

  • Packaging: I've only tried building/packaging locally since i have no knowledge of the CI inner workings.

  • "E2E Tests": I had to replace spectron with playwright since spectron is deprecated and lacks newer electron support. I had to disable some steps in the video/test script because I couldn't figure out how to correctly expand the correct subnodes in the tree (playwright strictly follows accessibility rules and won't allow click events on elements that are not visible). I.e for Livingroom/Lamp to be clickable, Livingroom must first be clicked. I made some progress, but currently it does not work with non-unique node Names, hence the disabled tests.

<span key="edge" className={this.props.classes.sourceEdge} data-test-topic={name}>

// disable this scenario for now until expandTopic is sorted out

export async function expandTopic(path: string, browser: Page) {

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

No branches or pull requests