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

Git holds xml instead of actual code #34

Open
syonip opened this issue Jan 18, 2016 · 5 comments
Open

Git holds xml instead of actual code #34

syonip opened this issue Jan 18, 2016 · 5 comments

Comments

@syonip
Copy link

syonip commented Jan 18, 2016

This makes it really hard to view diffs on the code.
Is there any way to make the actual class files be saved to git, instead of the xml representations?

Thanks,
Jonathan.

@adaptun
Copy link
Contributor

adaptun commented Jan 18, 2016

Hi Jonathan.

There is Caché class %Compiler.UDL.TextServices that supports export/import of classes in text format.

That would be major change in cache-tort-git to export in text format instead of XML. Currently all I do is call $system.OBJ.Export/$system.OBJ.Load for any (class, routine, inc) item. For text format I would have to handle different cases, depending on item type.

Given that Atelier (new Eclipse-based Studio) is already in beta testing, I don't think cache-tort-git will support export/import in text format.

Regards,
Alexander.

@adaptun
Copy link
Contributor

adaptun commented Jan 18, 2016

this is basically copy of #8

@syonip
Copy link
Author

syonip commented Jan 18, 2016

Thanks Alexander.
Can you recommend some workaround I could implement, let's say just for CLS files, to convert the xml to cls?

@adaptun
Copy link
Contributor

adaptun commented Jan 18, 2016

I don't think there is an easy workaround.

You might try to check simple source control class:
https://github.com/intersystems-ru/cache-source-control

It also uses XML as import/export format.

You might try to replace $$Export^%occXMLExport and $$Import^%occSAX(filename,"fv-d-l") to methods SetTextFromFile, GetTextAsFile of %Compiler.UDL.TextServices. But again -- you'll need to check that you are doing this only for classes

@syonip
Copy link
Author

syonip commented Jan 18, 2016

Thank you I will take a look.

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