synchronoss / cpo-api

Class Persistence Object (CPO) Application Programming Interface (API).
GNU Lesser General Public License v2.1
3 stars 3 forks source link

Can ConvertSQLToXML be added back in? #61

Closed sncr-github closed 3 years ago

sncr-github commented 4 years ago

https://github.com/synchronoss/cpo-api/commit/bb25d38e374a72ba87a95098b8013e1feb3e6593#diff-1dfd50ab45900d7be42b6ae2195d98c5a0773d9b13a83f4ba4a8fe0edc8bc149

Attempting to upgrade the ri-plugin... and this class is still used :).

BTW - Hi Dave... its Cas here. CPO still lives in the RI world.

berryware commented 4 years ago

This was only for conversion. You can do a conversion with an older version of the plugin. Or are you still using the database and converting on the fly for each build?

On 14 Oct, 2020, at 17:08 , sncr-github notifications@github.com wrote:

bb25d38#diff-1dfd50ab45900d7be42b6ae2195d98c5a0773d9b13a83f4ba4a8fe0edc8bc149 https://github.com/synchronoss/cpo-api/commit/bb25d38e374a72ba87a95098b8013e1feb3e6593#diff-1dfd50ab45900d7be42b6ae2195d98c5a0773d9b13a83f4ba4a8fe0edc8bc149 Attempting to upgrade the ri-plugin... and this class is still used :).

BTW - Hi Dave... its Cas here. CPO still lives in the RI world.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/synchronoss/cpo-api/issues/61, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABYRLQCKIAG5LS3P7S3NADSKYHOFANCNFSM4SRD7EWA.

sncr-github commented 4 years ago

Do you mean use an older version of maven (and associated dependencies)?

In general, I was trying to update to the latest version of various maven (3.6.1) plugins. I updated the cpo version from 5.2.0 to 5.2.2-SNAPSHOT

When using 5.2.1 I got the error... "The API of the mojo scanner is not compatible with this plugin version" because of references in cpo-plugin to AbstractMojo. Now that I look at it ConvertSQLToXML.java is one of the two classes that references AbstractMojo (the other is GenerateJavaSources.java). Ugh.

MojoEncryptionBase is the class I noticed that uses ConvertSQLToXML.

I'll look again tomorrow and get back to you. Thanks for the quick response.

berryware commented 3 years ago

@mbellomo and I talked about this and are not going to add this back in. All projects should have migrated to xml metadata by now