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

app-misc/mosquitto: bump to 1.6.2 #12037

Closed
wants to merge 1 commit into from
Closed

app-misc/mosquitto: bump to 1.6.2 #12037

wants to merge 1 commit into from

Conversation

oxr463
Copy link
Contributor

@oxr463 oxr463 commented May 18, 2019

Package-Manager: Portage-2.3.66, Repoman-2.3.12
Signed-off-by: Lucas Ramage [email protected]

@gentoo-bot
Copy link

Copyright policy change

Please note that on 2018-09-15 Trustees have approved new Gentoo copyright policy. All contributions made to Gentoo need to follow this policy. If you include the Signed-off-by line in your commit message, you indicate that you have read the policy and agree to its terms. For more detailed explanation, please see the new Gentoo copyright policy explained article.

Pull Request assignment

Submitter: @oxr463
Areas affected: ebuilds
Packages affected: app-misc/mosquitto

app-misc/mosquitto: @oxr463, @gentoo/proxy-maint

Linked bugs

No bugs to link found. If your pull request references any of the Gentoo bug reports, please add appropriate GLEP 66 tags to the commit message and request reassignment.


In order to force reassignment and/or bug reference scan, please append [please reassign] to the pull request title.

Docs: Code of ConductCopyright policy (expl.) ● DevmanualGitHub PRsProxy-maint guide

@gentoo-bot gentoo-bot added self-maintained The PR changes only packages that are maintained by the submitter (i.e. no need to ask anybody else) assigned PR successfully assigned to the package maintainer(s). labels May 18, 2019
Copy link
Member

@mgorny mgorny left a comment

Choose a reason for hiding this comment

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

distcc[54] ERROR: compile (null) on localhost failed
persist_read_test.c:6:10: fatal error: CUnit/CUnit.h: No such file or directory
 #include <CUnit/CUnit.h>
          ^~~~~~~~~~~~~~~
compilation terminated.
distcc[52] ERROR: compile persist_read_test.c on localhost failed
make[2]: *** [<builtin>: persist_read_test.o] Error 1

@oxr463
Copy link
Contributor Author

oxr463 commented May 23, 2019

distcc[54] ERROR: compile (null) on localhost failed
persist_read_test.c:6:10: fatal error: CUnit/CUnit.h: No such file or directory
 #include <CUnit/CUnit.h>
          ^~~~~~~~~~~~~~~
compilation terminated.
distcc[52] ERROR: compile persist_read_test.c on localhost failed
make[2]: *** [<builtin>: persist_read_test.o] Error 1

Ping @mgorny, please check again.

@mgorny
Copy link
Member

mgorny commented May 23, 2019

 * QA Notice: Files built without respecting LDFLAGS have been detected
 *  Please include the following list of files in your report:
 * /usr/bin/mosquitto_pub
 * /usr/bin/mosquitto_rr
 * /usr/bin/mosquitto_sub

@mgorny
Copy link
Member

mgorny commented May 23, 2019

RepoMan scours the neighborhood...
  RDEPEND.suspect               1
   app-misc/mosquitto/mosquitto-1.6.2.ebuild: 'test?' USE conditional in RDEPEND

@oxr463
Copy link
Contributor Author

oxr463 commented May 23, 2019

 * QA Notice: Files built without respecting LDFLAGS have been detected
 *  Please include the following list of files in your report:
 * /usr/bin/mosquitto_pub
 * /usr/bin/mosquitto_rr
 * /usr/bin/mosquitto_sub

How did you get this output?

I'm running this command to install:

FEATURES="test" ACCEPT_KEYWORDS="~amd64" USE="bridge" emerge app-misc/mosquitto
RepoMan scours the neighborhood...
  RDEPEND.suspect               1
   app-misc/mosquitto/mosquitto-1.6.2.ebuild: 'test?' USE conditional in RDEPEND

Oops, I'll move that into DEPEND directly.

@mgorny
Copy link
Member

mgorny commented May 24, 2019 via email

@oxr463

This comment has been minimized.

@oxr463
Copy link
Contributor Author

oxr463 commented May 27, 2019

Created an issue upstream, (See: eclipse/mosquitto#1294).

@oxr463
Copy link
Contributor Author

oxr463 commented May 27, 2019

@mgorny please review again. Thank you

@oxr463
Copy link
Contributor Author

oxr463 commented Jun 1, 2019

Ping @mgorny

app-misc/mosquitto/mosquitto-1.6.2.ebuild Outdated Show resolved Hide resolved
app-misc/mosquitto/mosquitto-1.6.2.ebuild Outdated Show resolved Hide resolved
app-misc/mosquitto/mosquitto-1.6.2.ebuild Outdated Show resolved Hide resolved
@oxr463
Copy link
Contributor Author

oxr463 commented Jun 7, 2019

@mgorny, I made the requested changes, however, the test-suite is failing for me due to a kernel upgrade, (I think?), eclipse/mosquitto#1307.

@mgorny
Copy link
Member

mgorny commented Jun 8, 2019

I get:

./11-message-expiry.py
1559979290: mosquitto version 1.6.2 starting
1559979290: Config loaded from 11-message-expiry.conf.
1559979290: Opening ipv4 listen socket on port 1888.
1559979290: Opening ipv6 listen socket on port 1888.
1559979290: New connection from ::1 on port 1888.
1559979290: Socket error on client <unknown>, disconnecting.
1559979292: New connection from 127.0.0.1 on port 1888.
1559979292: New client connected from 127.0.0.1 as subpub-qos0-test (p5, c0, k60).
1559979292: No will message specified.
1559979292: Sending CONNACK to subpub-qos0-test (1, 0)
1559979292: Sending PUBLISH to subpub-qos0-test (d0, q1, r0, m1, 'subpub/qos1', ... (8 bytes))
1559979292: Sending PUBLISH to subpub-qos0-test (d0, q1, r0, m2, 'subpub/qos1', ... (8 bytes))
1559979292: Sending PUBLISH to subpub-qos0-test (d0, q1, r0, m3, 'subpub/qos1', ... (8 bytes))
1559979292: Socket error on client subpub-qos0-test, disconnecting.
1559979292: mosquitto version 1.6.2 terminating
1559979292: Saving in-memory database to mosquitto-1888.db.

Maybe run it a few times to see if it's flakiness.

@oxr463
Copy link
Contributor Author

oxr463 commented Jun 13, 2019

I get:

./11-message-expiry.py
1559979290: mosquitto version 1.6.2 starting
1559979290: Config loaded from 11-message-expiry.conf.
1559979290: Opening ipv4 listen socket on port 1888.
1559979290: Opening ipv6 listen socket on port 1888.
1559979290: New connection from ::1 on port 1888.
1559979290: Socket error on client <unknown>, disconnecting.
1559979292: New connection from 127.0.0.1 on port 1888.
1559979292: New client connected from 127.0.0.1 as subpub-qos0-test (p5, c0, k60).
1559979292: No will message specified.
1559979292: Sending CONNACK to subpub-qos0-test (1, 0)
1559979292: Sending PUBLISH to subpub-qos0-test (d0, q1, r0, m1, 'subpub/qos1', ... (8 bytes))
1559979292: Sending PUBLISH to subpub-qos0-test (d0, q1, r0, m2, 'subpub/qos1', ... (8 bytes))
1559979292: Sending PUBLISH to subpub-qos0-test (d0, q1, r0, m3, 'subpub/qos1', ... (8 bytes))
1559979292: Socket error on client subpub-qos0-test, disconnecting.
1559979292: mosquitto version 1.6.2 terminating
1559979292: Saving in-memory database to mosquitto-1888.db.

Maybe run it a few times to see if it's flakiness.

I tried it again in a virtual machine running Linux kernel 4.19, and I'm getting the same error as before so I don't think it has anything to do with my kernel version.

Package-Manager: Portage-2.3.66, Repoman-2.3.12
Signed-off-by: Lucas Ramage <[email protected]>
@oxr463
Copy link
Contributor Author

oxr463 commented Jun 22, 2019

@mgorny, please review. I removed all failing tests from the Makefile.

@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2019-06-22 02:30 UTC
Newest commit scanned: f31fff8
Status: ✅ good

No issues found

@mgorny
Copy link
Member

mgorny commented Jun 22, 2019

Do you have any clue why they fail? Is upstream aware?

@oxr463
Copy link
Contributor Author

oxr463 commented Jun 22, 2019

Do you have any clue why they fail? Is upstream aware?

I'm not 100% sure why they are failing, but I created an issue upstream; I'll continue troubleshooting from there.

@oxr463 oxr463 deleted the app-misc/mosquitto-1.6.2 branch June 22, 2019 10:35
@oxr463 oxr463 restored the app-misc/mosquitto-1.6.2 branch June 22, 2019 10:46
@oxr463 oxr463 deleted the app-misc/mosquitto-1.6.2 branch February 24, 2020 16:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
assigned PR successfully assigned to the package maintainer(s). self-maintained The PR changes only packages that are maintained by the submitter (i.e. no need to ask anybody else)
Projects
None yet
4 participants