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

Merge DNSSD into main Poco #4323

Closed
wants to merge 33 commits into from
Closed

Merge DNSSD into main Poco #4323

wants to merge 33 commits into from

Conversation

soroshsabz
Copy link
Contributor

ITNOA

Merge DNSSD into main Poco project

fixes #2208

obiltschnig and others added 26 commits October 5, 2016 10:24
@obiltschnig
Initial commit
56e7388
@obiltschnig
initial commit
a30ef28
@obiltschnig
added README.md
68b4d76
@obiltschnig
Update README.md
a38083e
Add top level CMakeLists like another project in POCO framework. see p…
6754a4b 
…ocoproject#1
@soroshsabz
Add CMakeLists to Avahi and Bonjour. (see pocoproject#1)
b942a10
@soroshsabz
Missing changing in top level CMakeLists correct. (see pocoproject#1)
fe1f856
@soroshsabz
Add samples CMakeLists. (see pocoproject#1)
79ccb8d
@soroshsabz
Add temporary cmake find module for Avahi and Bonjour in cmake direct…
64f415a 
…ory. (see pocoproject#1)
@soroshsabz
Add mandatory requirement diff for POCO framework to DNSSD cmake can …
f12ab8f 
…be work correctly. (see pocoproject#1)
@soroshsabz
Update README.md
1d4e33e 
Add cmake build way.
@soroshsabz
Update README.md
a6cdd1c 
Minor change.
@soroshsabz
Update README.md
8f78c1c 
Removed ambiguous sentence.
@soroshsabz
Moved files
674b83a
@soroshsabz
Add cmake modules
682e68d
@soroshsabz
Add cmake modules
e660d4e
@soroshsabz
Merge remote-tracking branch 'sub/master' into devel
6344a44
@soroshsabz
Remove modules
a73861b
@soroshsabz
Correct linux cmake ci.
4af6687
@soroshsabz
Exclude DNSSD from macos, windows.
95047c2
@soroshsabz
Merge branch 'devel' into devel
d4fc7cd
@soroshsabz
Update CMakeLists.txt
6d04f50
@soroshsabz
Remove unused gitignore
1e5027a
@soroshsabz
Merge branch 'devel' of https://github.com/soroshsabz/poco into devel
93ff2ed
@aleks-f
Merge branch 'devel' into devel
fae860b
@soroshsabz
Merge branch 'pocoproject:devel' into devel
767bad3
@soroshsabz soroshsabz mentioned this pull request Dec 5, 2023
Closed
@soroshsabz
Copy link
Contributor Author

@aleks-f @matejk as you say, I create new PR, so please start CI

thanks

aleks-f
aleks-f requested changes Dec 6, 2023
View reviewed changes
Copy link
Member

@aleks-f aleks-f left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we need only vs_90, vs_160 and vs_170 projects here (see #4274). all others should be deleted, and vs_160 and vs_170 generated with progen

@teksturi teksturi mentioned this pull request Dec 15, 2023
Merged
@geirhei
Copy link

geirhei commented Feb 2, 2024

Any progress on this @soroshsabz? The library is very useful, would be great to get it into POCO.

@soroshsabz
Copy link
Contributor Author

@aleks-f I am done

please run CI

thanks

@soroshsabz soroshsabz requested a review from aleks-f February 25, 2024 17:16
@soroshsabz
Copy link
Contributor Author

@matejk

@matejk
Copy link
Contributor

matejk commented Feb 26, 2024

I don't know why only a few of the CI jobs were executed and not all of them.

aleks-f
aleks-f requested changes Feb 28, 2024
View reviewed changes
DNSSD/Bonjour/Bonjour_vs100.vcxproj Outdated Show resolved Hide resolved
DNSSD/samples/DNSSDBrowser/DNSSDBrowser_WEC2013_vs120.vcxproj.filters Outdated Show resolved Hide resolved
ActiveRecord/ActiveRecord_vs170.vcxproj Outdated Show resolved Hide resolved
DNSSD/DNSSD_vs90.vcproj Show resolved Hide resolved
@soroshsabz
Copy link
Contributor Author

@aleks-f I resolve all your notes

please run CI

@soroshsabz soroshsabz requested a review from aleks-f February 29, 2024 09:47
@aleks-f aleks-f mentioned this pull request Mar 1, 2024
Merged
@aleks-f
Copy link
Member

aleks-f commented Mar 1, 2024

#4479

@aleks-f aleks-f closed this Mar 1, 2024
@soroshsabz soroshsabz mentioned this pull request Mar 28, 2024
Closed
@soroshsabz
Copy link
Contributor Author

soroshsabz commented Mar 28, 2024 •
edited
Loading

@aleks-f

I am happy to reopen this PR and merge that PR

thanks

@soroshsabz soroshsabz mentioned this pull request Mar 28, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aleks-f aleks-f Awaiting requested review from aleks-f

Assignees

@matejk matejk

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Merge Poco DNSSD to Poco
5 participants
@soroshsabz @geirhei @matejk @aleks-f @obiltschnig