njh / mqtt-sn-tools

Command line tools written in C for the MQTT-SN (MQTT for Sensor Networks) protocol
MIT License
195 stars 84 forks source link

Subscribe to multiple topic names and multiple pre-defined topic IDs. #18

Closed MichalFoksa closed 7 years ago

MichalFoksa commented 8 years ago

Command line options -t and -T may occur multiple times with mqtt-sn-sub.

njh commented 7 years ago

Hi @MichalFoksa,

Please could you fix the merge conflicts, so that this PR can be accepted?

Thanks,

nick.

MichalFoksa commented 7 years ago

Hi @njh

Give me some time to catch up with project changes, PLS.

Michal

MichalFoksa commented 7 years ago

Hi Nicholas,

I cannot pass tests in Travis with this pull request. Do not know what the error mean.

Can you PLS help me with it?

Michal

njh commented 7 years ago

Ok, I will take a look. If there are no merge conflicts in Git, then I can fix any tests.

Sent from my phone

On 16 Aug 2017, at 17:34, Michal Foksa notifications@github.com wrote:

Hi Nicholas,

I cannot pass tests in Travis with this pull request. Do not know what the error mean.

Can you PLS help me with it?

Michal

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

MichalFoksa commented 7 years ago

There are no conflicts.

I just dropped test to avoid simultaneous topic name and topic id subscription, while it is possible now.

Thank you, Michal

njh commented 7 years ago

I fixed the failing test in 495806b9481bccd39d6ff093c10b40edc47e51df

Hopefully Travis will go green again now