-
Notifications
You must be signed in to change notification settings - Fork 43
added automation for z/OS open tools #124
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
base: master
Are you sure you want to change the base?
added automation for z/OS open tools #124
Conversation
Signed-off-by: aashish-khatri <[email protected]>
xtanion
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @aashish-khatri, PTAL at my comments below, There are a few things you might need to change.
Also, note that you need to crop the name of the package and remove the suffix port. For eg: emacsport changes to emacs. As you can see in the file created by @arshPratap.
Thanks.
| repo_info = {"packageName":repo["name"],"description":repo["description"],"version":""} | ||
| zOS_list.append(repo_info) | ||
|
|
||
| with open('ZOSOpenTools.json','w') as file: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This will save the json file in the pwd, we may want to move it to ./distro_data/ instead
| print(f"Saved!\nfilename: {q}") | ||
|
|
||
| def zOSOpenTools(): | ||
| url = "https://api.github.com/users/ZOSOpenTools/repos" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you use this URL, you'll not find all repository names in your response as you can see in your ZOSOpentools.json. Refer to this.
| zOS_list.append(repo_info) | ||
|
|
||
| with open('ZOSOpenTools.json','w') as file: | ||
| json.dump(zOS_list,file,indent=2) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Saving it directly will erase all the previous data in the json. You should compare the new data with the old one and append it to the end of the file.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We actually do just clobber old versions of the files 😆 so this is in keeping with the rest of the tool. I think it's OK, but something we should address more holistically. Can you create an issue for addressing this tool-wide?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pleia2 I think its OK for now , we need to keep in mind the changes we are going to add in the future for the SQL based database design and I dont think creating any issue will be required for now .
pleia2
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks to @xtanion for the reviews, aside from the one I commented on, these should be addressed before we're ready to merge.
|
@aashish-khatri In addition to above, you'll also want to add ZOSOpenTools.json to src/config/supported_distros.py |
In issue #49 a case was made that we have several sources for open source software being developed for IBM z/OS and it now makes sense to split up the repositories for easier maintenance and improved accuracy. This commit splits out the following: - Rocket Software into ZOS_Rocket_Software.json - Everything from github.com/IBM/IBM-Z-zOS/ into IBM_Z_zOS.json - Everything maintained by the Open Mainframe Project into ZOS_OMP.json Once this split is completed, we will work on updates to these files. ZOS_Software_List.json remains as a generic place to put any software that doesn't neatly fit into another category, but I have removed all tools related to the z/OS Open Tools project, since it's very outdated and there's another effort in progress to complete adding these tools into their own source file (see: openmainframeproject/software-discovery-tool#124). Signed-off-by: Elizabeth K. Joseph <[email protected]>
In issue #49 a case was made that we have several sources for open source software being developed for IBM z/OS and it now makes sense to split up the repositories for easier maintenance and improved accuracy. This commit splits out the following: - Rocket Software into ZOS_Rocket_Software.json - Everything from github.com/IBM/IBM-Z-zOS/ into IBM_Z_zOS.json - Everything maintained by the Open Mainframe Project into ZOS_OMP.json Once this split is completed, we will work on updates to these files. ZOS_Software_List.json remains as a generic place to put any software that doesn't neatly fit into another category, but I have removed all tools related to the z/OS Open Tools project, since it's very outdated and there's another effort in progress to complete adding these tools into their own source file (see: openmainframeproject/software-discovery-tool#124). Signed-off-by: Elizabeth K. Joseph <[email protected]>
In issue #49 a case was made that we have several sources for open source software being developed for IBM z/OS and it now makes sense to split up the repositories for easier maintenance and improved accuracy. This commit splits out the following: - Rocket Software into ZOS_Rocket_Software.json - Everything from github.com/IBM/IBM-Z-zOS/ into IBM_Z_zOS.json - Everything maintained by the Open Mainframe Project into ZOS_OMP.json Once this split is completed, we will work on updates to these files. ZOS_Software_List.json remains as a generic place to put any software that doesn't neatly fit into another category, but I have removed all tools related to the z/OS Open Tools project, since it's very outdated and there's another effort in progress to complete adding these tools into their own source file (see: openmainframeproject/software-discovery-tool#124). Signed-off-by: Elizabeth K. Joseph <[email protected]>
In issue #49 a case was made that we have several sources for open source software being developed for IBM z/OS and it now makes sense to split up the repositories for easier maintenance and improved accuracy. This commit splits out the following: - Rocket Software into ZOS_Rocket_Software.json - Everything from github.com/IBM/IBM-Z-zOS/ into IBM_Z_zOS.json - Everything maintained by the Open Mainframe Project into ZOS_OMP.json Once this split is completed, we will work on updates to these files. ZOS_Software_List.json remains as a generic place to put any software that doesn't neatly fit into another category, but I have removed all tools related to the z/OS Open Tools project, since it's very outdated and there's another effort in progress to complete adding these tools into their own source file (see: openmainframeproject/software-discovery-tool#124). Signed-off-by: Elizabeth K. Joseph <[email protected]>
In issue #49 a case was made that we have several sources for open source software being developed for IBM z/OS and it now makes sense to split up the repositories for easier maintenance and improved accuracy. This commit splits out the following: - Rocket Software into ZOS_Rocket_Software.json - Everything from github.com/IBM/IBM-Z-zOS/ into IBM_Z_zOS.json - Everything maintained by the Open Mainframe Project into ZOS_OMP.json Once this split is completed, we will work on updates to these files. ZOS_Software_List.json remains as a generic place to put any software that doesn't neatly fit into another category, but I have removed all tools related to the z/OS Open Tools project, since it's very outdated and there's another effort in progress to complete adding these tools into their own source file (see: openmainframeproject/software-discovery-tool#124). Signed-off-by: Elizabeth K. Joseph <[email protected]>
* Split out z/OS package data In issue #49 a case was made that we have several sources for open source software being developed for IBM z/OS and it now makes sense to split up the repositories for easier maintenance and improved accuracy. This commit splits out the following: - Rocket Software into ZOS_Rocket_Software.json - Everything from github.com/IBM/IBM-Z-zOS/ into IBM_Z_zOS.json - Everything maintained by the Open Mainframe Project into ZOS_OMP.json Once this split is completed, we will work on updates to these files. ZOS_Software_List.json remains as a generic place to put any software that doesn't neatly fit into another category, but I have removed all tools related to the z/OS Open Tools project, since it's very outdated and there's another effort in progress to complete adding these tools into their own source file (see: openmainframeproject/software-discovery-tool#124). Signed-off-by: Elizabeth K. Joseph <[email protected]>
|
I have made the changes as requested in this PR. Link |
Fixes to add packages compatible with z/OS in response to issue #110. @arshPratap @pleia2 please take a look.