I would like to merge Jason's pull request quickly, because it's clear it's
needed before we can make a release. I'll probably do that on Friday, giving
folks a chance to comment. If you find that APRS-IS connections are still
wonky with or without IPv6, please comment on the pull request directly.
I would also like to merge my pull request #20 (fixing ax25 warnings) in
fairly short order, but don't want to do it until a few folks have chimed
in. That code change has an impact on everyone who is using Linux kernel
AX.25 networking, which I only use occasionally with Xastir. So it would
be best if a bunch of folks who use it regularly could try out the change
and make sure it breaks nothing. And if you *were* seeing kernel warnings
about the obsolute "PF_INET, SOCK_PACKET" usage in syslog, it would be good
to know that they're gone now. Again, if you get a chance to try it out,
commenting on the pull request at github would be appreciated.
After both pull requests are merged, I'll probably push out a new stable
release at github. That probably won't happen until at least 10 Feb.
--
Tom Russo KM5VY SAR502 DM64ux http://www.swcp.com/~russo/
Tijeras, NM QRPL#1592 K2#398 SOC#236 http://kevan.org/brain.cgi?DDTNM
echo "prpv_a'rfg_cnf_har_cvcr" | sed -e 's/_/ /g' | tr [a-m][n-z] [n-z][a-m]
--- Begin Message ---
We weren't breaking out of connect() loop on success. This could either lead to
the address with the lowest priority being used, or possibly failing to connect
at all.
Make sure the channel FD is set to -1 when we close the socket. Otherwise we
could try to close it again when the same FD number is being used by another
interface.
You can view, comment on, or merge this pull request online at:
https://github.com/Xastir/Xastir/pull/21
-- Commit Summary --
* Break from net connect() loop on success.
* Set interface channel to -1 when we close socket.
-- File Changes --
M src/interface.c (4)
-- Patch Links --
https://github.com/Xastir/Xastir/pull/21.patch
https://github.com/Xastir/Xastir/pull/21.diff
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/Xastir/Xastir/pull/21
--- End Message ---
_______________________________________________
Xastir-dev mailing list
[email protected]
http://xastir.org/mailman/listinfo/xastir-dev
