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

Please release a new tag containing logrotate #2

Closed
pysan3 opened this issue Feb 8, 2023 · 6 comments
Closed

Please release a new tag containing logrotate #2

pysan3 opened this issue Feb 8, 2023 · 6 comments

Comments

@pysan3
Copy link

pysan3 commented Feb 8, 2023

Hi @shawnfeng0,

Thanks for the amazing tool :)

I was using the log_rotate program everyday and just found out that you migrated that tool to this repository.

However, the latest release tag - v0.4.0 does not seem to include the log_rotate tool yet.

May I kindly ask you to create a new tag that contains your latest code since I have a code to manually build it with the latest tag?

Thanks in advance.

@shawnfeng0
Copy link
Owner

shawnfeng0 commented Feb 8, 2023

Yes, the log_rotate tool has been migrated to ulog. It is now called logrotate, and its command line parameter names are connected by dashes.

Now the logrotate inside ulog is stable and available, and I will release a new version number later.

Since log_rotate doesn't have any github star/watch/fork, I thought there was no risk in deleting the standalone log_rotate, but sorry to bother you.

Have you seen the instructions for migrating to ulog internal tools on the readme homepage of log_rotate before?

@pysan3
Copy link
Author

pysan3 commented Feb 8, 2023

Now the logrotate inside ulog is stable and available, and I will release a new version number later.

Thank you for your quick response! And no worries, I at least had a local copy which auto updates.

its command line parameter names are connected by dashes.

Ah, I missed this. Thanks for pointing out!

Have you seen the instructions for migrating to ulog internal tools on the readme homepage of log_rotate before?

My local README copy says like this. Is this what you meant?

Please use https://github.com/shawnfeng0/ulog/tree/main/tools/logrotate as an alternative

I am only using the command line interface so I am not really concerned about migrating c code or anything but would be ok if I have the executable.

After cmake and make, The previous build/log_rotate is equivalent of build/tools/logrotate/logrotate right?

@shawnfeng0
Copy link
Owner

My local README copy says like this. Is this what you meant?

Please use https://github.com/shawnfeng0/ulog/tree/main/tools/logrotate as an alternative

Yes!

I am only using the command line interface so I am not really concerned about migrating c code or anything but would be ok if I have the executable.

After cmake and make, The previous build/log_rotate is equivalent of build/tools/logrotate/logrotate right?

Yes! v0.4.1 has been released, note that the command line parameter names have changed a bit. You can check it with logrotate --help.

@pysan3
Copy link
Author

pysan3 commented Feb 8, 2023

Thank you so much.

However, I have one problem with v0.4.1.

It compiles without any problems on my Arch machine but when I switch to an Ubuntu 20.04 machine, I get undefined reference to pthread_create error. Probably this issue on Stackoverflow

I have no knowledge about c / c++ but I think we need to add pthread in the tools/logrotate/CMakeLists.txt? Just like mentioned in [examples](https://github.com/shawnfeng0/ulog/tree/main/examples)/CMakeLists.txt ?

at: @shawnfeng0

@shawnfeng0
Copy link
Owner

It compiles without any problems on my Arch machine but when I switch to an Ubuntu 20.04 machine, I get undefined reference to pthread_create error. Probably this issue on Stackoverflow

I have no knowledge about c / c++ but I think we need to add pthread in the tools/logrotate/CMakeLists.txt? Just like mentioned in [examples](https://github.com/shawnfeng0/ulog/tree/main/examples)/CMakeLists.txt ?

Yes, I've added it and urgently released version 0.4.2.
I used to have errors when I used the old version of gcc/clang, but now the gcc/clang on my machine are all up-to-date (Arch Linux), maybe the latest compiler has made some changes, it did not report an error, in order to ensure compatibility I added it.

@pysan3
Copy link
Author

pysan3 commented Feb 9, 2023

Thank you so much!

Works like a charm ;)

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

No branches or pull requests

2 participants