larshp / abapGitServer

Git server implemented in ABAP
MIT License
64 stars 25 forks source link

Short dump program ZAGS_MIGRATION_01 #102

Closed treisinger closed 6 years ago

treisinger commented 6 years ago

Hello,

I tried to run program ZAGS_MIGRATION_01. I get a dump because of following assert statement.

image

In my case all objects have already a repo and sha1 assignment. Not sure it is an error or done by purpose. It might be better just to send information message that there is nothing to do. Or shouldn't this program called directly?

Best regards, Tapio

larshp commented 6 years ago

If column REPO in ZAGS_OBJECTS contains blank/empty values this program must be run manually. In your case it looks like everything is okay.

Yeah, the program should be changed to give a "ok" message instead of dumping

Also note the calls CALL METHOD ('\PROGRAM=ZABAPGIT\CLASS=LCL_GIT_PACK')=>decode_tree will dump

The program is around a year old, so if you installed abapGitServer after this, then everything is okay

treisinger commented 6 years ago

Ok, thanks.

larshp commented 6 years ago

lets keep it open, will add the ok message, plus fix the dump when calling CALL METHOD ('\PROGRAM=ZABAPGIT\CLASS=LCL_GIT_PACK')=>decode_tree