search

kieker-monitoring / kieker

Kieker's main repository
Apache License 2.0
70 stars 41 forks source link

[KIEKER-1733] Functionality and format of script generateGraph.cmd #2106

Closed rju closed 2 weeks ago

rju commented 2 weeks ago

JIRA Issue: KIEKER-1733 Functionality and format of script generateGraph.cmd Original Reporter: Andre van Hoorn

The bin directory includes a new script generateGraph.cmd.

It is not clear what this script does. It actually seems to read files from one hard-coded directory and overwrite some files in the same directory.

If this should go into the release, the script should be aligned with the other .bat scripts (file extension, parameterized directories, .sh version).

Moreover, a hard-coded reference to the Kieker Jar is used.

CC'ing christian.wulf

rju commented 2 weeks ago

author André van Hoorn -- Thu, 3 May 2018 11:10:44 +0200

Decided to remove the script from release 1.14. Please re-add after clean-up.

As a reference, here are the contents of the file (additionally visible in the Git change set, of course).

avhAndrè van Hoorn-ThinkPad-T450s:~/git_work/kieker/bin (1.14)*$ cat generateGraph.cmd 
rem java -cp %kieker% teetime.framework.ConfigurationStarter kieker.analysisteetime.config.DependencyGraphConfiguration 

echo Execute from within Kieker's root directory.

set kieker=build\libs\kieker-1.14-SNAPSHOT-emf.jar
set kiekerLogDir=kieker-analysis/test-resources/kieker-20170805-132418-9229368724068-UTC--KIEKER
set graphOutputDir=%kiekerLogDir%
java -cp %kieker% kieker.analysisteetime.config.DependencyGraphConfiguration %kiekerLogDir% %graphOutputDir%
rju commented 2 weeks ago

author Unknown -- Fri, 4 May 2018 21:33:11 +0200

The script executes the first analysis based on TeeTime. Indeed, it requires some effort to make it ready for the public.

rju commented 2 weeks ago

author avanhoorn -- Thu, 6 May 2021 09:06:23 +0200

Deleted