Should (can?) we update the entire Pjon library from upstream (instead of this single change)?

Should (can?) we update the entire Pjon library from upstream (instead of this single change)?

@mfalkvidd Hi Mikael,
Yes, upgrading the PJON Library is a good idea and should be done. And yes, the upgrade can potentially introduce new problems.

I personally don't use MySensors with PJON. So I can't judge if the new version really works with MySensors. So I think a PR with the upgrade to the latest PJON version should be requested by a PJON user who can test this with his use case.

@gryzli133 is it possible for you to create a PR with an update to the latest PJON version?

I personally find the CAN bus extension of @AdamSlowik PR #1488 very interesting and would be happy if it will be included in MySensors.

Unfortunately Toll Gate does not accept the pull request as described. And also several other PRs, e.g. @Tico06 with PR #1520, currently have the same problem and are blocked.

Therefore, my suggestion is to first merge this PR which (hopefully) do not changes the current PJON functionality, so that the other PRs are no longer blocked. Afterwards, the PR of the PJON Library upgrade can be placed and tested at leisure.

BR Immo

Hi There,

Thanks a lot @virtual-maker for the fix and @mfalkvidd for the merge. I just commited let's see what will happen.

Thx and br

Eric.