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.

Sign up for GitHub

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

Jump to bottom

Future3/better logging #2139

Closed
wants to merge 20 commits into from
Closed

Future3/better logging #2139

wants to merge 20 commits into from

Conversation

pabera
Copy link
Collaborator

@pabera pabera commented Dec 8, 2023

For better readability, this PR uses descriptive function names for logging instead of repetitive code

print_lc "This message will be logged to both console and logfile"
print_c  "This message will only be logged to the console"
log      "This message will only be logged to the logfile"

pabera and others added 20 commits December 3, 2023 13:18
@pabera
Bump libzmq version to 4.3.5 for armv7
dbfd58a
@pabera
Allow download of prebuild libzmq-v4.3.5 for Docker
01ec0bc
@pabera
Merge branch 'future3/develop' into future3/bump-libzmq
935a6d4
@pabera
Remove unnecessary build-essentials from apt-get in Dockerfile
8ea49cf
@pabera
Fix a few typos
a32b771
@pabera
Use wget to download pre-built libzmq from github
78eb0c4
@pabera
Remove unused constants
5a875b8
@pabera
Make variable local to its function
54681da
@pabera
Install webapp from release build if Node.JS download is aborted
ade2894
@pabera
Use GH release downloads for webapp
c3d0897
@pabera
Bugfix: wrong variable assignment
3ffe8a3
@pabera
Static release download url
bdb1dd1
@pabera
Fix version funtion return value
caf4a55
@AlvinSchiller
fix local libzmq build
554c0ed 
fix folder handling ZMQ_TMP_DIR.
remove libsodium (use internal encryption lib tweetnacl).
add cpu_count.
add --disable-Werror.
add sudo to make install.
update log messaages.
@AlvinSchiller
add CI test for local libzmq build
435bd6f
@AlvinSchiller
update var names and docs
23cf06c
@AlvinSchiller
add platform in scope and description
55b1c9e
@pabera
Merge pull request #1 from AlvinSchiller/future3/bump-libzmq
93212d0 
Fix libzmq local build
@pabera
Introduce proper logging functions
46b360d
@pabera
Fix a few minor things
042ed9a
@pabera pabera closed this Dec 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@pabera @AlvinSchiller