Skip to content

Update READ with more stuff #1

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

Closed
wants to merge 1 commit into from
Closed

Update READ with more stuff #1

wants to merge 1 commit into from

Conversation

jrossi
Copy link
Member

@jrossi jrossi commented Nov 19, 2013

More stuff is cool

More stuff is cool
@jrossi
Copy link
Member Author

jrossi commented Nov 19, 2013

This looks ok - could you add unite test

@ghost ghost assigned jbcheng Nov 19, 2013
@jrossi
Copy link
Member Author

jrossi commented Nov 19, 2013

not needed

@jrossi jrossi closed this Nov 19, 2013
@jrossi jrossi reopened this Nov 19, 2013
@jrossi jrossi closed this Nov 19, 2013
@jrossi jrossi reopened this Nov 19, 2013
@jrossi jrossi closed this Nov 21, 2013
@ghost ghost assigned ddpbsd Jan 24, 2014
ddpbsd pushed a commit that referenced this pull request Mar 4, 2014
ddpbsd pushed a commit that referenced this pull request May 13, 2015
ddpbsd pushed a commit that referenced this pull request Sep 29, 2015
@herrwamm herrwamm mentioned this pull request Dec 28, 2015
atomicturtle pushed a commit that referenced this pull request Apr 28, 2016
Minor fixes to case / grammar in README.md
@mziadh mziadh mentioned this pull request Oct 14, 2016
ddpbsd pushed a commit that referenced this pull request Jan 5, 2017
This PR fixes the following Windows-related build issues:

S_ISLNK usage
S_ISLNK isn't a supported call on Windows due to the lack of filesystem links like they exist on Unix-based systems. Other parts of the code properly check if WIN32 is defined, however, one piece was overlooked. This is fixed in one of the commits in this PR.

MINGW_HAS_SECURE_API Incorrectly Used
The MINGW_HAS_SECURE_API was used incorrectly and broke the Windows build - the correct usage is to define this to '1' instead of just defining it, as evident in the mingw headers.

Add Windows-specific output to .gitignore
Some of the build output files aren't added to the .gitignore, causing a potential source tree pollution when committing to the repository. Most notably, there was no exclusion for *.exe and *.dll files, while there was an exclusion for .so files. This discrepancy has been corrected.
ddpbsd pushed a commit that referenced this pull request Apr 30, 2018
atomicturtle pushed a commit that referenced this pull request Jan 6, 2021
* Fix compiling and suspend warnings in read_journald.c

* Add READING_JOURNAL #define for debug printing in logcollector

* Add 'journald' as allowed XML config parameter for 'logformat'
atomicturtle pushed a commit that referenced this pull request Aug 9, 2023
add parental rule id for rule 52000
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