rlenferink closed pull request #22: migrated readdir_r (deprecated) to readdir.
URL: https://github.com/apache/celix/pull/22
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:
As this is a foreign pul
jsjolen opened a new pull request #28: Feat 6 filter gtest
URL: https://github.com/apache/celix/pull/28
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL
jsjolen commented on issue #28: Feat 6 filter gtest
URL: https://github.com/apache/celix/pull/28#issuecomment-467598601
Oops.
This is an automated message from the Apache Git Service.
To respond to the message, please log on G
jsjolen closed pull request #28: Feat 6 filter gtest
URL: https://github.com/apache/celix/pull/28
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above
jsjolen edited a comment on issue #28: Feat 6 filter gtest
URL: https://github.com/apache/celix/pull/28#issuecomment-467598601
Oops. Thatäs not supposed to be here.
This is an automated message from the Apache Git Service.
To
rbulter opened a new pull request #29: Update Cmake files to fix OSX build
URL: https://github.com/apache/celix/pull/29
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Git
rbulter opened a new pull request #29: Update Cmake files to fix OSX build
URL: https://github.com/apache/celix/pull/29
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Git
rbulter commented on issue #29: Update Cmake files to fix OSX build
URL: https://github.com/apache/celix/pull/29#issuecomment-492772935
I fixed the OSX build
This is an automated message from the Apache Git Service.
To respond
rbulter closed pull request #29: Update Cmake files to fix OSX build
URL: https://github.com/apache/celix/pull/29
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub an
rbulter closed pull request #29: CELIX-462 Update Cmake files to fix OSX build
URL: https://github.com/apache/celix/pull/29
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
rbulter opened a new pull request #30: Feature/celix 462 fix osx build
URL: https://github.com/apache/celix/pull/30
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
rbulter closed pull request #30: Feature/celix 462 fix osx build
URL: https://github.com/apache/celix/pull/30
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and us
rbulter opened a new pull request #31: Feature/celix 462 fix osx build
URL: https://github.com/apache/celix/pull/31
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
pnoltes merged pull request #31: Feature/celix 462 fix osx build
URL: https://github.com/apache/celix/pull/31
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and us
rbulter opened a new pull request #32: Merge pull request #1 from apache/develop
URL: https://github.com/apache/celix/pull/32
Merge Develop
This is an automated message from the Apache Git Service.
To respond to the message, p
rbulter closed pull request #32: Merge pull request #1 from apache/develop
URL: https://github.com/apache/celix/pull/32
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Git
rbulter commented on issue #32: Merge pull request #1 from apache/develop
URL: https://github.com/apache/celix/pull/32#issuecomment-506022266
Merge apace helix
This is an automated message from the Apache Git Service.
To respo
rbulter opened a new pull request #32: Merge pull request #1 from apache/develop
URL: https://github.com/apache/celix/pull/32
Merge Develop
This is an automated message from the Apache Git Service.
To respond to the message, p
rbulter closed pull request #32: Merge pull request #1 from apache/develop
URL: https://github.com/apache/celix/pull/32
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Git
dhbfischer opened a new pull request #33: Added HTTP/websocket admin with
civetweb as server
URL: https://github.com/apache/celix/pull/33
This is an automated message from the Apache Git Service.
To respond to the message, p
rbulter opened a new pull request #34: Feature/add tcp pubsub endpoint
URL: https://github.com/apache/celix/pull/34
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
rlenferink commented on issue #34: Feature/add tcp pubsub endpoint
URL: https://github.com/apache/celix/pull/34#issuecomment-509091689
Hi @rbulter. Thanks for your contribution of a TCP pubsubadmin to Celix.
However, it looks like the tests for the TCP pubsubadmin are failing:
https://tr
rlenferink commented on a change in pull request #34: Feature/add tcp pubsub
endpoint
URL: https://github.com/apache/celix/pull/34#discussion_r300936011
##
File path: bundles/pubsub/pubsub_admin_tcp/CMakeLists.txt
##
@@ -0,0 +1,52 @@
+# Licensed to the Apache Software Foun
rlenferink commented on a change in pull request #34: Feature/add tcp pubsub
endpoint
URL: https://github.com/apache/celix/pull/34#discussion_r300937458
##
File path: bundles/pubsub/pubsub_admin_tcp/src/pubsub_tcp_topic_receiver.c
##
@@ -0,0 +1,745 @@
+/**
+ *Licensed to t
rlenferink commented on a change in pull request #34: Feature/add tcp pubsub
endpoint
URL: https://github.com/apache/celix/pull/34#discussion_r300939510
##
File path: bundles/pubsub/pubsub_admin_tcp/src/pubsub_tcp_topic_receiver.c
##
@@ -0,0 +1,745 @@
+/**
+ *Licensed to t
rlenferink commented on a change in pull request #34: Feature/add tcp pubsub
endpoint
URL: https://github.com/apache/celix/pull/34#discussion_r300939596
##
File path: bundles/pubsub/pubsub_admin_tcp/src/pubsub_tcp_topic_receiver.c
##
@@ -0,0 +1,745 @@
+/**
+ *Licensed to t
rlenferink commented on a change in pull request #34: Feature/add tcp pubsub
endpoint
URL: https://github.com/apache/celix/pull/34#discussion_r300939510
##
File path: bundles/pubsub/pubsub_admin_tcp/src/pubsub_tcp_topic_receiver.c
##
@@ -0,0 +1,745 @@
+/**
+ *Licensed to t
rlenferink opened a new pull request #7: Converted site to Hugo
URL: https://github.com/apache/celix-site/pull/7
The pull request contains the Celix site converted from Jekyll to Hugo. This
is described on the [mailing
list](https://lists.apache.org/thread.html/1e83c6fc0f1bab00d42867ecd2ff
netlify[bot] commented on issue #7: Converted site to Hugo
URL: https://github.com/apache/celix-site/pull/7#issuecomment-510947939
Deploy preview for *celix-staging* ready!
Built with commit 1e44b19fd08ac4878e860785e7b147d34608b885
https://deploy-preview-7--celix-staging.netlif
rlenferink merged pull request #7: Converted site to Hugo
URL: https://github.com/apache/celix-site/pull/7
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use t
rlenferink commented on issue #33: Added HTTP/websocket admin with civetweb as
server
URL: https://github.com/apache/celix/pull/33#issuecomment-511373473
Hi @dhbfischer, after voting on
[dev@celix](https://lists.apache.org/thread.html/b14b2c4221d40d0b610de738f6682b24de0e3d179ac13ad0f1b5df7
rlenferink merged pull request #33: Added HTTP/websocket admin with civetweb as
server
URL: https://github.com/apache/celix/pull/33
This is an automated message from the Apache Git Service.
To respond to the message, please
dhbfischer opened a new pull request #35: Http admin
URL: https://github.com/apache/celix/pull/35
A few improvements for HTTP admin
This is an automated message from the Apache Git Service.
To respond to the message, please lo
rlenferink merged pull request #35: Http admin
URL: https://github.com/apache/celix/pull/35
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to
rbulter closed pull request #34: Feature/add tcp pubsub endpoint
URL: https://github.com/apache/celix/pull/34
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and us
rbulter opened a new pull request #36: Add pubsub_admin_tcp
URL: https://github.com/apache/celix/pull/36
Add PubSub_admin_tcp
This is an automated message from the Apache Git Service.
To respond to the message, please log on t
rlenferink commented on issue #36: Add pubsub_admin_tcp
URL: https://github.com/apache/celix/pull/36#issuecomment-513105079
Hi @rbulter thanks for your contribution. I've reviewed this contribution
and it looks good to me.
Because this is a substantial contribution (a new pubsub admi
dhbfischer opened a new pull request #37: Framework bugfixes
URL: https://github.com/apache/celix/pull/37
Fixed segmentation faults with dangling service trackers and service
registrations
This is an automated message from th
rlenferink merged pull request #37: Framework bugfixes
URL: https://github.com/apache/celix/pull/37
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
rlenferink commented on issue #36: Add pubsub_admin_tcp
URL: https://github.com/apache/celix/pull/36#issuecomment-515703038
Hi @rbulter, after voting on
[dev@celix](https://lists.apache.org/thread.html/0f9ba172a6eee909173b70d8552b58a9fa4928f1f067d5b1b3d134ed@%3Cdev.celix.apache.org%3E)
and
rlenferink merged pull request #36: Add pubsub_admin_tcp
URL: https://github.com/apache/celix/pull/36
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
UR
rlenferink opened a new pull request #38: Added support for custom msg ids in
descriptor
URL: https://github.com/apache/celix/pull/38
This PR adds support for custom message ids to message descriptors. If no
message ID is specified, the default 'utils_stringHash' function is used.
---
dhbfischer opened a new pull request #39: Added pubsub admin websocket
URL: https://github.com/apache/celix/pull/39
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
rlenferink commented on issue #39: Added pubsub admin websocket
URL: https://github.com/apache/celix/pull/39#issuecomment-519526567
Hi @dhbfischer. Thanks for yet another code donation. I've reviewed this
contribution and it looks good to me. It is building and tests are running as
well (1
rbulter opened a new pull request #40: Fix tcp seg fault
URL: https://github.com/apache/celix/pull/40
This pull request fixes:
- tcp admin segfault when unloading service
- make the tcp examples work.
@pnoltes I have fixed some issues pubsub topology admin.
that fixes crashes
rbulter closed pull request #40: Fix tcp seg fault
URL: https://github.com/apache/celix/pull/40
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL abov
rbulter opened a new pull request #41: - Fix TCP example.
URL: https://github.com/apache/celix/pull/41
- Remove TCP crash after unloading tcp admin service
(Problem in pubsub topology admin)
- Enable pubsub sending after unloading pubsub tcp admin
(Problem in pubsub topology adm
rbulter commented on issue #41: - Fix TCP example.
URL: https://github.com/apache/celix/pull/41#issuecomment-519690426
This pull request fixes:
- tcp admin segfault when unloading service
- make the tcp examples work.
@pnoltes I have fixed some issues pubsub topology admin.
t
rlenferink commented on a change in pull request #41: - Fix TCP example.
URL: https://github.com/apache/celix/pull/41#discussion_r312465533
##
File path: bundles/pubsub/pubsub_admin_tcp/src/pubsub_tcp_topic_sender.c
##
@@ -198,7 +195,7 @@ pubsub_tcp_topic_sender_t *pubsub_t
rbulter commented on a change in pull request #41: - Fix TCP example.
URL: https://github.com/apache/celix/pull/41#discussion_r312608715
##
File path: bundles/pubsub/pubsub_admin_tcp/src/pubsub_tcp_topic_sender.c
##
@@ -198,7 +195,7 @@ pubsub_tcp_topic_sender_t *pubsub_tcpT
rlenferink commented on issue #41: - Fix TCP example.
URL: https://github.com/apache/celix/pull/41#issuecomment-521562407
Changes to the PSA look good to me. I did check out the changes and all our
existing examples are still running fine and the `pubsub_publisher_tcp` &
`pubsub_subscriber
pnoltes merged pull request #41: - Fix TCP example.
URL: https://github.com/apache/celix/pull/41
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL abo
rlenferink opened a new pull request #42: Added check for serializer type when
connecting endpoints
URL: https://github.com/apache/celix/pull/42
Updated the pubsub admins to check for serializer type when connecting
endpoints.
--
rlenferink commented on issue #42: Added check for serializer type when
connecting endpoints
URL: https://github.com/apache/celix/pull/42#issuecomment-522503309
_Waiting for ICLA from Michiel_
This is an automated message fro
rlenferink merged pull request #42: Added check for serializer type when
connecting endpoints
URL: https://github.com/apache/celix/pull/42
This is an automated message from the Apache Git Service.
To respond to the message,
rlenferink commented on issue #42: Added check for serializer type when
connecting endpoints
URL: https://github.com/apache/celix/pull/42#issuecomment-522737517
_ICLA received from Michiel_
This is an automated message from t
idzardh opened a new pull request #43: Develop
URL: https://github.com/apache/celix/pull/43
Implemented avpr parsing, generation of dyn_type en updating the serializer
tests for the generated dyn_type
This is an automated mes
rlenferink merged pull request #39: Added pubsub admin websocket
URL: https://github.com/apache/celix/pull/39
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and us
rlenferink commented on issue #43: Added avpr parsing and serialization
URL: https://github.com/apache/celix/pull/43#issuecomment-524793081
Can you update the `bundles/pubsub/pubsub_serializer_json` as well? =>
https://github.com/apache/celix/blob/develop/bundles/pubsub/pubsub_serializer_js
idzardh commented on issue #43: Added avpr parsing and serialization
URL: https://github.com/apache/celix/pull/43#issuecomment-526080299
Added json_serializer and fixed differences between build types. All tests
passed
This i
rlenferink merged pull request #43: Added avpr parsing and serialization
URL: https://github.com/apache/celix/pull/43
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHu
pnoltes opened a new pull request #44: Initial setup for a GitHub Ci flow
URL: https://github.com/apache/celix/pull/44
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitH
pnoltes closed pull request #44: Initial setup for a GitHub Ci flow
URL: https://github.com/apache/celix/pull/44
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
rlenferink merged pull request #38: Added support for custom msg ids in
descriptor
URL: https://github.com/apache/celix/pull/38
This is an automated message from the Apache Git Service.
To respond to the message, please log
idzardh opened a new pull request #45: Fix for curl dependency
URL: https://github.com/apache/celix/pull/45
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
rlenferink commented on a change in pull request #45: Fix for curl dependency
URL: https://github.com/apache/celix/pull/45#discussion_r320109195
##
File path: CMakeLists.txt
##
@@ -49,6 +52,33 @@ ELSE ()
set(CMAKE_EXE_LINKER_FLAGS "-pthread ${CMAKE_EXE_LINKER_FLAGS}")
rlenferink commented on a change in pull request #45: Fix for curl dependency
URL: https://github.com/apache/celix/pull/45#discussion_r320109770
##
File path: cmake/celix_project/AddOpenSSL.cmake
##
@@ -0,0 +1,39 @@
+# Licensed to the Apache Software Foundation (ASF) under
rlenferink commented on issue #45: Fix for curl dependency
URL: https://github.com/apache/celix/pull/45#issuecomment-527325878
Note for the one approving this PR: please squash and merge due to the 42
commits in this PR.
This
rlenferink edited a comment on issue #45: Fix for curl dependency
URL: https://github.com/apache/celix/pull/45#issuecomment-527325878
Note for the one merging this PR: please squash and merge due to the 42
commits in this PR.
idzardh commented on a change in pull request #45: Fix for curl dependency
URL: https://github.com/apache/celix/pull/45#discussion_r320111222
##
File path: cmake/celix_project/AddOpenSSL.cmake
##
@@ -0,0 +1,39 @@
+# Licensed to the Apache Software Foundation (ASF) under one
rlenferink merged pull request #45: Fix for curl dependency
URL: https://github.com/apache/celix/pull/45
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
idzardh opened a new pull request #46: Develop
URL: https://github.com/apache/celix/pull/46
Allow for openssl dependencies with find_package
This is an automated message from the Apache Git Service.
To respond to the message,
pnoltes commented on issue #46: Develop
URL: https://github.com/apache/celix/pull/46#issuecomment-528506427
Could you update the FindOpenSSL.cmake with the following:
find_path(OPENSSL_INCLUDE_DIR ssl.h crypto.h
/usr/include/openssl
/usr/local/include/openssl
rlenferink commented on issue #46: Develop
URL: https://github.com/apache/celix/pull/46#issuecomment-528513361
@pnoltes remember to squash and merge (57 commits in this PR) and fill in an
appropriate commit message
This is an
idzardh commented on issue #46: Develop
URL: https://github.com/apache/celix/pull/46#issuecomment-528517593
@pnoltes your suggestion indeed fixed the build for OSX, thank you for the
help!
This is an automated message from th
pnoltes merged pull request #46: Develop
URL: https://github.com/apache/celix/pull/46
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to
dhbfischer opened a new pull request #47: Feature/http admin improvements
URL: https://github.com/apache/celix/pull/47
Improved deletion of http admin services
This is an automated message from the Apache Git Service.
To respo
dhbfischer opened a new pull request #48: Feature/pubsub tcp admin fix
URL: https://github.com/apache/celix/pull/48
Fixed tcp admin port extraction from string
Fix for setting bind ip in sender
This is an automated message
rlenferink commented on a change in pull request #48: Feature/pubsub tcp admin
fix
URL: https://github.com/apache/celix/pull/48#discussion_r322751349
##
File path: bundles/pubsub/pubsub_admin_tcp/src/pubsub_tcp_handler.c
##
@@ -451,7 +451,8 @@ void pubsub_tcpHandler_setUrl
pnoltes merged pull request #47: Feature/http admin improvements
URL: https://github.com/apache/celix/pull/47
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and us
pnoltes merged pull request #48: Feature/pubsub tcp admin fix
URL: https://github.com/apache/celix/pull/48
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use t
dhbfischer opened a new pull request #49: Feature/openssl linking
URL: https://github.com/apache/celix/pull/49
Added configurable openssl library for linking
This is an automated message from the Apache Git Service.
To respond
coveralls commented on issue #49: Feature/openssl linking
URL: https://github.com/apache/celix/pull/49#issuecomment-530246713
[](https://coveralls.io/builds/25646784)
Coverage increased (+0.03%) to 58.569% when pulling
pnoltes merged pull request #49: Feature/openssl linking
URL: https://github.com/apache/celix/pull/49
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
UR
rbulter opened a new pull request #50: Feature/fix tcp transmissions
URL: https://github.com/apache/celix/pull/50
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub an
pnoltes opened a new pull request #51: Feature/handle hooks in registry
URL: https://github.com/apache/celix/pull/51
Fixes a race condition issue with adding/removing service listener hooks
(service trackers).
Also fixes an mem issue in the json serialiser and refactors the pubsub
rlenferink commented on a change in pull request #50: Feature/fix tcp
transmissions
URL: https://github.com/apache/celix/pull/50#discussion_r324850011
##
File path: CMakeLists.txt
##
@@ -38,8 +38,10 @@ set(ENABLE_MORE_WARNINGS OFF)
IF (ANDROID)
set(CMAKE_C_FLAGS "-D_
rlenferink merged pull request #50: Feature/fix tcp transmissions
URL: https://github.com/apache/celix/pull/50
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and u
rlenferink merged pull request #51: Feature/handle hooks in registry
URL: https://github.com/apache/celix/pull/51
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub an
idzardh opened a new pull request #52: Added parsing of internal types
URL: https://github.com/apache/celix/pull/52
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
rlenferink merged pull request #52: Added parsing of internal types
URL: https://github.com/apache/celix/pull/52
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
idzardh opened a new pull request #53: CELIX-468
URL: https://github.com/apache/celix/pull/53
Renamed constants to celix_constants (including ifdef)
Changed all includes to use new file name
@pnoltes the issue mentions that some variables should be prefixed with
CELIX, however all var
pnoltes commented on issue #53: CELIX-468
URL: https://github.com/apache/celix/pull/53#issuecomment-533250446
> Renamed constants to celix_constants (including ifdef)
> Changed all includes to use new file name
> @pnoltes the issue mentions that some variables should be prefixed with
pnoltes merged pull request #53: CELIX-468
URL: https://github.com/apache/celix/pull/53
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go
idzardh opened a new pull request #54: CELIX-468 Two extra test-cases
URL: https://github.com/apache/celix/pull/54
Apparently these are not compiled in any of the test scenarios on the build
server, caught them during further testing
pnoltes merged pull request #54: CELIX-468 Two extra test-cases
URL: https://github.com/apache/celix/pull/54
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
rbulter opened a new pull request #55: Feature/refactor tcp
URL: https://github.com/apache/celix/pull/55
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
rlenferink merged pull request #55: Feature/refactor tcp
URL: https://github.com/apache/celix/pull/55
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
UR
pnoltes opened a new pull request #56: Adds a experimental dir to the Celix
project and moves some bundles
URL: https://github.com/apache/celix/pull/56
Currently more stable and unstable bundles are in the same dir, to make a
more clear distiction what is consider really unstable/experimen
rlenferink merged pull request #56: Adds a experimental dir to the Celix
project and moves some bundles
URL: https://github.com/apache/celix/pull/56
This is an automated message from the Apache Git Service.
To respond to the
1 - 100 of 2506 matches
Mail list logo