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.

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

[Docs] Add note about sendfile.h removal #16234

Merged
merged 1 commit into from
Feb 10, 2022
Merged

[Docs] Add note about sendfile.h removal #16234

merged 1 commit into from
Feb 10, 2022

Conversation

VirtualTim
Copy link
Collaborator

See #16200.
Files was removed in #14248.

@VirtualTim
Copy link
Collaborator Author

Crap I thought prefixing the PR with [Docs] would skip the tests.

@kripken
Copy link
Member

kripken commented Feb 10, 2022

Optimally it would skip the non-doc tests, yeah. But we do have build-docs which should run on docs builds in general... not sure there's a good way to avoid the wasted work.

@sbc100 did you have any more thoughts on resolving this?

@sbc100
Copy link
Collaborator

sbc100 commented Feb 10, 2022

Optimally it would skip the non-doc tests, yeah. But we do have build-docs which should run on docs builds in general... not sure there's a good way to avoid the wasted work.

@sbc100 did you have any more thoughts on resolving this?

Trying to figure out which test to run on a given PR is fairly non-trivial... we could try to do it based on magic comments in the PR description or based on which files are touched, but I'm not sure its worth it. WDYT?

@sbc100 sbc100 merged commit b1bbec1 into main Feb 10, 2022
@sbc100 sbc100 deleted the VirtualTim-doco branch February 10, 2022 19:59
bas524 added a commit to bas524/poco that referenced this pull request Dec 8, 2023
add NotImplemented exception for unsupported platforms
exculude <sys/sendfile.h> for POCO_EMSCRIPTEN, because https://
github.com/emscripten-core/emscripten/pull/16234
matejk pushed a commit to pocoproject/poco that referenced this pull request Dec 11, 2023
* add sendfile method for streamsocket

* add mswsock.lib to the project files and templates

* remove /DPOCO_NO_AUTOMATIC_LIBS for cmake windows build

* merge from upstream

* merge from upstream

* fix code stile
add NotImplemented exception for unsupported platforms
exculude <sys/sendfile.h> for POCO_EMSCRIPTEN, because https://
github.com/emscripten-core/emscripten/pull/16234

* add iostream include for std::cout

* fix compilation for emscripten (wrap sendfile)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants