Skip to content

Comments

Update Cirrus CI FreeBSD images#181

Merged
ericcurtin merged 1 commit intoinotify-tools:masterfrom
emaste:patch-1
Jun 21, 2023
Merged

Update Cirrus CI FreeBSD images#181
ericcurtin merged 1 commit intoinotify-tools:masterfrom
emaste:patch-1

Conversation

@emaste
Copy link
Contributor

@emaste emaste commented Jun 21, 2023

FreeBSD 13.2 and 12.4 are the currently supported releases

FreeBSD 13.2 and 12.4 are the currently supported releases
@ericcurtin
Copy link
Member

Thank you @emaste

Just a question, for the next version of inotifytools, I am changing the codebase from a C one, to be a codebase that supports a restricted set of C++ that doesn't require extra libraries (no requirement on libstdc++) in order to achieve safer code. This adds some basic C++ features such as destructors, classes, C++ style structs, more type safety etc. Do you have any feedback on this before I push?

To give you an idea of what it would look like:

master...tmp_test

@ericcurtin
Copy link
Member

Merging passed FreeBSD builds

@ericcurtin ericcurtin merged commit 239122d into inotify-tools:master Jun 21, 2023
@emaste
Copy link
Contributor Author

emaste commented Jun 21, 2023

This adds some basic C++ features such as destructors, classes, C++ style structs, more type safety etc. Do you have any feedback on this before I push?

I'm not sure what weight my answer should have, but I think this is fine to do. We've started making more use of C++ in the FreeBSD base system for some tools, for similar reasons. Of course a library has more concerns but I think using the limited set of C++ features you mention is sensible.

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.

2 participants