Bug 247884 - New port: www/gwsocket Simple WebSocket Server
Summary: New port: www/gwsocket Simple WebSocket Server
Status: New
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: freebsd-ports-bugs (Nobody)
Depends on:
Reported: 2020-07-10 05:51 UTC by Daniel Morante
Modified: 2020-08-02 10:31 UTC (History)
1 user (show)

See Also:

SVN diff (13.72 KB, patch)
2020-07-10 05:51 UTC, Daniel Morante
no flags Details | Diff
SVN diff (13.66 KB, patch)
2020-07-10 07:45 UTC, Daniel Morante
no flags Details | Diff
SVN diff (13.88 KB, patch)
2020-07-11 04:43 UTC, Daniel Morante
no flags Details | Diff
SVN diff (13.86 KB, patch)
2020-07-11 04:50 UTC, Daniel Morante
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel Morante 2020-07-10 05:51:38 UTC
Created attachment 216353 [details]
SVN diff

gwsocket is a simple, standalone, language-agnostic, RFC6455 compliant WebSocket Server, written in C. It sits between your application and the client's browser, giving fast bidirectional communication between these two with ease and flexibility. 

More info at: http://gwsocket.io.

This port supports the ability to run multiple instances using *.conf files.
Comment 1 Daniel Morante 2020-07-10 07:45:26 UTC
Created attachment 216357 [details]
SVN diff

Removed UTF8 option (not supported).
Comment 2 daniel.engberg.lists 2020-07-10 22:38:46 UTC
GNU_CONFIGURE and INSTALL_TARGET seem out of place, portlint doesn't complain?


CFLAGS and LDFLAGS can't be replace by USES= localbase ?

Any specific reason why you're using a fairly dynamic approach to pkg-plist?


Does it pass Poudreire?

Best regards,
Comment 3 Daniel Morante 2020-07-11 04:42:27 UTC
(In reply to daniel.engberg.lists from comment #2)

Thank you for the feedback.  

Portlint only warned about the absolute path names for "/var/run/.." and "/var/log". This Makefile was derived from an existing port and I missed double checking that.   Regardless, I moved the GNU_CONFIGURE line to more sensible placement and removed INSTALL_TARGET since it seems to have no effect.

I also made a few changes to the pkg-list according to your suggestions, and got rid of the CFLAGS and LDFLAGS lines since those are also unnecessary.

While making the above changed I discovered an error with the way the OPENSSL option was being applied and fixed it.

The port passes Poudreire:
Comment 4 Daniel Morante 2020-07-11 04:43:10 UTC
Created attachment 216372 [details]
SVN diff

Updated svn diff with corrections.
Comment 5 Daniel Morante 2020-07-11 04:50:54 UTC
Created attachment 216373 [details]
SVN diff

Upload correct patch.
Comment 6 daniel.engberg.lists 2020-07-19 10:23:00 UTC
LGTM, nice work!
Comment 7 daniel.engberg.lists 2020-08-02 10:31:20 UTC
Poudriere testport OK 12.1-RELEASE (amd64)