-
Notifications
You must be signed in to change notification settings - Fork 321
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
change internal API to support "spk" and "deb" #158
base: master
Are you sure you want to change the base?
Conversation
Hey, @crea-doo could you please give some context for these changes? They are quite big. Also please check the pull request. It does not allow for an automatic merge. |
Hi, thanks for the quick response ;-) So basically the So regarding context of the changes:
If I have time I'll also add a short description similar to the descriptions in the I hope I didn't miss anything... |
Hmmm. While I can see the overlap I am more inclined to say: let's revamp the API so you use |
I think this is too specific to become part of jdeb. That's great if you were able to reuse jdeb to support this format, but I think it's outside the scope of jdeb. |
Sorry to hear that :-( |
The |
+1 for the SPK generation (just the Feature :-)) @crea-doo it would be great if you could share your achievements with us (see our Issue openhab/openhab1-addons#1989). Keep up the good work! Thomas E.-E. |
What is here the state, when it goes on? |
I extended jdeb with a new Maven Mojo and a Ant task that supports the creation of packages/archives for Synoloy DiskStations in the spk format. I tried to reuse as much as possible the already existing functions and utilies and also refactored and generalized where necessary.
Please review the changes and let me know, if you need me to change anything. I really would like to see this functionality in jdeb.