dreamyguy / gitlogg

💾 🧮 🤯 Parse the 'git log' of multiple repos to 'JSON'
MIT License
130 stars 27 forks source link

Ensure that there are no paralellization hickups with xargs #16

Closed Inventitech closed 7 years ago

Inventitech commented 7 years ago

Refs #10

Inventitech commented 7 years ago

Hi @dreamyguy,

I see there is some good stuff coming, but maybe you can accept this PR in the meantime to remove the parallelization hickups that introducing xargs caused.

dreamyguy commented 7 years ago

Got the following error:

Wallaces-iMac:gitlogg Wallace$ bash ./scripts/gitlogg-generate-log.sh
Info: Calculating in 7 thread(s)
Generating git log for all 4 repositories located at './_repos/*/'. This might take a while!
Outputting git.git
Outputting core.git
Outputting linux.git
Outputting odoo.git
./scripts/output-intermediate-gitlog.sh: line 30: _tmp/gitlogg.tmp.part.: No such file or directory
./scripts/output-intermediate-gitlog.sh: line 30: _tmp/gitlogg.tmp.part.: No such file or directory
./scripts/output-intermediate-gitlog.sh: line 30: _tmp/gitlogg.tmp.part.: No such file or directory
./scripts/output-intermediate-gitlog.sh: line 30: _tmp/gitlogg.tmp.part.: No such file or directory
cat: _tmp/gitlogg.tmp.part.*: No such file or directory
rm: _tmp/gitlogg.tmp.part.*: No such file or directory
The file _tmp/gitlogg.tmp generated in: 0s
Parsing JSON output...
JSON output parsed in: 7ms
The file _output/gitlogg.json was saved!
Wallaces-iMac:gitlogg Wallace$

Maybe there's some left-over code lying around? This line also seem displaced.

dreamyguy commented 7 years ago

I'll merge the patch for the xargs problem, but I'm changing the base at my end to develop instead of master branch. I do this everywhere else and might as well start doing it here. 🎸

Inventitech commented 7 years ago

Oops, I think something went wrong while rebasing :-).

Inventitech commented 7 years ago

Have a look now, please.

dreamyguy commented 7 years ago

I've decided to merge to a dedicated branch instead, to keep develop relese-able. Will be testing it asap.