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

Add a script to output contents of log files within a folder #253

Closed
ihhub opened this issue Nov 11, 2018 · 8 comments · Fixed by #673
Closed

Add a script to output contents of log files within a folder #253

ihhub opened this issue Nov 11, 2018 · 8 comments · Fixed by #673
Labels
good first issue An issue is perfectly suitable for first comers. A very minimal knowledge of the project is required help wanted We need a help improvement Upgrade existing feature
Milestone

Comments

@ihhub
Copy link
Owner

ihhub commented Nov 11, 2018

In Travis-CI script (.travis) for dynamic analysis we output valgrind results into log files. It's done by MemChecker itself. Currently we output results of log file with fixed name:

- cat "${TRAVIS_BUILD_DIR}/build/Testing/Temporary/MemoryChecker.2.log" > out && cat out

The main attention is on number 2 which is actually hardcoded. It's not a perfect solution.
To avoid this we need to create a script which would find all files with .log extension and output all contents of these files.

@ihhub ihhub added improvement Upgrade existing feature Travis-CI labels Nov 11, 2018
@ihhub ihhub added this to the Release milestone Nov 11, 2018
@yannlabou yannlabou added help wanted We need a help good first issue An issue is perfectly suitable for first comers. A very minimal knowledge of the project is required labels Nov 20, 2018
@manekenpix
Copy link
Contributor

Is this one still available?

@manekenpix
Copy link
Contributor

I know Hacktoberfest is about to be over, but it's not for that, I just like the project and I'd like to help.

@ihhub
Copy link
Owner Author

ihhub commented Oct 26, 2019

Hi @manekenpix, yes, you could take this issue.

@manekenpix
Copy link
Contributor

Thank you, I have a couple of questions.

To avoid this we need to create a script which would find all files with .log extension and output all contents of these files.

  • Output all contents of these files on screen or to another file?
  • Do you want to rename the file with a 2 to something else?

@manekenpix
Copy link
Contributor

I'm still interested in this one. I recently asked about #397, but I think I'll work on this one first since I asked first. @ihhub did you read my comment?

@ihhub
Copy link
Owner Author

ihhub commented Nov 19, 2019

Hi @manekenpix , I'm sorry for not replying as I missed your previous message. We need to output the content of files to command line/screen.
Right now value 2 is just hardcoded value which means nothing. Instead of this we exactly need to read all files in directory without such hardcoded values.

@manekenpix
Copy link
Contributor

I'll give this one a go for this coming Hacktoberfest, if that's ok (sorry about not having worked on it before)

@ihhub
Copy link
Owner Author

ihhub commented Sep 25, 2020

Sure!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue An issue is perfectly suitable for first comers. A very minimal knowledge of the project is required help wanted We need a help improvement Upgrade existing feature
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants