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

correct mosquitto.h return code documentation #1690

Merged
merged 1 commit into from May 20, 2020
Merged

Conversation

martingkelly
Copy link
Contributor

@martingkelly martingkelly commented May 18, 2020

We currently erroneously claim that mosquitto_lib_init cannot fail,
while it can fail on Windows, if WSAStartup fails in net__init. Correct
this.

  • Have you signed the Eclipse Contributor Agreement, using the same email address as you used in your commits?
  • Do each of your commits have a "Signed-off-by" line, with the correct email address? Use "git commit -s" to generate this line for you.
  • If you are contributing a bugfix, is your work based off the fixes branch?
  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you successfully run make test with your changes locally?
    "make test" is failing on my machine, but it fails identically on the fixes branch (on ./02-subscribe-qos1.py). As this changes only documentation, I highly doubt that's the issue.
    Test failure:
./02-subscribe-qos1.py c/02-subscribe-qos1-async2.test
connect_async failed: Connection refused
Traceback (most recent call last):
  File "./02-subscribe-qos1.py", line 47, in <module>
    (conn, address) = sock.accept()
  File "/usr/lib/python3.8/socket.py", line 292, in accept
    fd, addr = self._accept()
socket.timeout: timed out
make[2]: *** [Makefile:36: c] Error 1
make[2]: Leaving directory '/home/martin/ws/mosquitto/test/lib'
make[1]: *** [Makefile:11: test] Error 2
make[1]: Leaving directory '/home/martin/ws/mosquitto/test'
make: *** [Makefile:75: test] Error 2

We currently erroneously claim that mosquitto_lib_init cannot fail,
while it can fail on Windows, if WSAStartup fails in net__init. Correct
this.

Signed-off-by: Martin Kelly <[email protected]>
@ralight
Copy link
Contributor

ralight commented May 20, 2020

Thank you.

Out of interest, what system are you running on? All the tests pass fine here, so it would be good to find the difference.

@ralight ralight merged commit d103174 into eclipse:fixes May 20, 2020
@martingkelly martingkelly deleted the doc-fix branch May 20, 2020 16:50
@martingkelly
Copy link
Contributor Author

I'm running on Linux, and I never hit this issue at all. I found this out from code analysis rather than testing.

@ralight
Copy link
Contributor

ralight commented May 20, 2020

Sorry I wasn't sufficiently clear, I was referring to the ./02-subscribe-qos1.py c/02-subscribe-qos1-async2.test failure.

@martingkelly
Copy link
Contributor Author

Ah, ok. My system is debian bullseye (testing):

martin@columbia:~$ cat /etc/os-release 
PRETTY_NAME="Debian GNU/Linux bullseye/sid"
NAME="Debian GNU/Linux"
ID=debian
HOME_URL="https://www.debian.org/"
SUPPORT_URL="https://www.debian.org/support"
BUG_REPORT_URL="https://bugs.debian.org/"

Copy link

@Alexbeckii Alexbeckii left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 9, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants