Created attachment 200109 [details]
An alternative plugin for bitlbee that allows connections to the discord chat service. Does not require use of libpurple. My first port! Comment and constructive criticism welcome!
Great job for a first timer Arthur, well done!
At a cursory view, the changes look OK.
What you'll want to do from here is get your QA game on, so you can be confident about any this and future contributions before you submit them, to pick up issues early and often and to save time going back and forth on Bugzilla issues for issues tooling can objectively identify.
Check out the following section in the Porters Handbook, which goes over the details and instructions for the tools we use to test ports. In particular (and in summary) you'll want to use:
- DEVELOPER=yes in /etc/make.conf (and poudriere's make.conf's)
If you have any questions, get stuck or need anything else, hit us up at #freebsd-ports on freenode IRC.
Nice, thank you.
I compiled and installed the port on a fresh 12.0 machine with the bitlbee package installed using pkg.
Port compiles, installs and works great.
(In reply to Kubilay Kocak from comment #1)
Hi, thanks for the feedback! I had already done a portlint and a poudriere test run for my current arch, AMD64 12-0-RELEASE before submitting the patch, however was about to start testing for AMD64 11.2-RELEASE, the previous production release, and 12.0 and 11.2 releases for i386. It takes quite a while here as I'm doing all this in a virtual machine, I know, not the best environment for development! Thanks also for the reminder to enable DEVELOPER="yes", too!
Is this the test procedure, test to ensure the port runs on the current production releases for both AMD64 and i386?
(In reply to Michiel van Baak from comment #2)
Hi, glad I could help! I'm still considering this alpha, however happy that it works for you. I'm also running this from my port on my own machine here, too.
(In reply to Arthur Pirika from comment #3)
VM's are perfectly fine (I do mine in a virtualbox CURRENT guest), and its good to know you've run through portlint/poudriere. For future issues, its always good to mention this explicitly, so there's no questions or ambiguity, like:
portlint: OK (looks fine.)
testport: OK (poudriere: <versions, archs> tested)
Unless the software has a test suite, and the port has TEST_DEPENDS (if necessary) and a test: target to run the tests, poudriere (in various architecture/version jails) basically tests everything up to and including packaging (fetch, configure build, install), but not runtime, which is why test suite integration is fantastic if upstream provides them. You can mention this in your issues, like:
maketest: OK (XXX of YYY tests: PASS)
For some mode 101/cheatsheets/guidelines on Porting/Bugzilla/Workflow:
QA update. The following testing has now been done on this port.
make check-orphans: OK
make stage: OK
make package, as root: OK
make package, as user: OK
make install, in running system: OK
make deinstall, in running system: OK
portlint: Trivial warnings, no errors.
Poudriere runs: all successfull for the following architecture/version combos.
i386 /11.2-release, i386/12.0-release
Note: I have not tested amd64/11.2-release, as this port takes nearly 5 hours for me to compile, thanks to a dependency on llvm60 that's pulled in during the build process for *each* build! Also, I'm confident that 11.2-release on amd64 will work, as this is where I began the project.
Hope port committers find this useful!
Comment on attachment 200109 [details]
+USES= autoreconf gettext-runtime libtool pkgconfig
I would have it something like:
USES= autoreconf gettext-runtime libtool pkgconfig
For one, it keeps the USES* together, second typically suppose to be a new line after USES, GH_* can be put right below USE_GITHUB.
(In reply to Nathan from comment #7)
By USES, I meant USES* block
Created attachment 200142 [details]
new diff, with suggested changes to placement of variables.
(In reply to Arthur Pirika from comment #6)
> Hope port committers find this useful!
Yes, thank you!
A commit references this bug:
Date: Fri Jan 25 08:04:02 UTC 2019
New revision: 491132
New port: irc/bitlbee-discord
Bitlbee plugin to allow connections to the discord chat service.
A more lightweight alternative to using bitlbee compiled with
Submitted by: Arthur Pirika <email@example.com>
Reviewed by: koobs, firstname.lastname@example.org