Merge lp:~mbruzek/charms/trusty/ibm-xl-fortran/readme-update into lp:~ibmcharmers/charms/trusty/ibm-xl-fortran/trunk

Proposed by Matt Bruzek
Status: Needs review
Proposed branch: lp:~mbruzek/charms/trusty/ibm-xl-fortran/readme-update
Merge into: lp:~ibmcharmers/charms/trusty/ibm-xl-fortran/trunk
Diff against target: 205 lines (+85/-80)
1 file modified
README.md (+85/-80)
To merge this branch: bzr merge lp:~mbruzek/charms/trusty/ibm-xl-fortran/readme-update
Reviewer Review Type Date Requested Status
sunitha Pending
Review via email: mp+276817@code.launchpad.net

Description of the change

I found the README.md file incomplete and hard to follow.

For example it must be clear this is a Power PC only charm so I added a section near the top that made this very clear.

Also the existing README.md suggested editing the config.yaml file and users can not do that if they deploy this charm from the Charm Store. So I removed that section from the deployment steps.

This merge proposal does not fix all the problems with the README.md file! Some of the configuration uses underscores (_) and some configuration options use hypens (-) as word separators. The code, config.yaml and the README.md should be fixed to normalize on one method.

Please consider this merge proposal.

To post a comment you must log in.
52. By Matt Bruzek

Fixing grammer errors in the README update.

Unmerged revisions

52. By Matt Bruzek

Fixing grammer errors in the README update.

51. By Matt Bruzek

Updating the readme to fix issues that I found in the review.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'README.md'
2--- README.md 2015-09-30 06:58:52 +0000
3+++ README.md 2015-11-05 21:19:01 +0000
4@@ -1,88 +1,95 @@
5-Charm for IBM XL FORTRAN for Linux 15.1.2, Evaluation
6-
7-
8-Overview
9---------
10-
11-
12-IBM XL FORTRAN for Linux V15.1.2, Evaluation
13-
14-IBM XL FORTRAN for Linux, V15.1.2 is a standards-based, high-performance Fortran compiler with advanced optimization and performance-tuning features.
15-
16-For details on IBM XL FORTRAN, as well as information on purchasing the trial version,
17-please visit: [Trial Download] [trial-xlf].
18-
19-To get more details about the product and useful additional resources, please visit the product page
20-[Product Page] [product-page] and [IBM Knowledge Center] [ibm-xlf-info]
21-
22+# IBM XL Fortran
23+
24+IBM XL FORTRAN for Linux, V15.1.2 is a standards-based, high-performance
25+Fortran compiler with advanced optimization and performance-tuning features.
26+
27+For details on IBM XL FORTRAN, as well as information on purchasing the trial
28+version, please visit: [Trial Download][trial-xlf].
29+
30+To get more details about the product and useful additional resources, please
31+visit the [Product Page][product-page] and [IBM Knowledge Center][ibm-xlf-info]
32+
33+### This charm only runs on Power architecture (ppc64le)!
34
35 Usage
36 -----
37
38-Download your 60 day trial version for IBM XL Fortran for Linux and upload it to your apache
39-repository. To download and install the code contained in this charm you must agree to the IBM license.
40-The License files (`LicenseAgreement.pdf` and `license.pdf` ) can be found under the `files/archives` folder
41-after extraction of IBM Fortran compiler software is successful.
42-
43-Note: This charm will only work on power systems.
44-
45-# Creating the repositories in Apache
46-Run below steps with admin access
47-
48-Install the required packages
49- apt-get install dpkg-dev apache2
50-
51-Create a repository on Apache server under your Apache repository directory.
52-
53- cd <apache-repository-directory>
54- mkdir ibm-xlf
55-
56-Place the downloaded IBM XL FORTRAN for Linux packages in the above directory and Configure Apache server to allow access to the repository.
57-
58-Configure config.yaml with the repository url
59-
60- Example : `fortran_url: "http://<server-name>/<REPOSITORY_DIR>/fortran_url"`
61-
62- Verify repository is set correctly otherwise charm might throw errors.
63-
64-
65+Download your 60 day trial version for IBM XL Fortran for Linux and upload it
66+to your apache repository. To download and install the code contained in this
67+charm you must agree to the IBM license. The License files
68+(`LicenseAgreement.pdf` and `license.pdf` ) can be found under the
69+`files/archives` folder after extraction of IBM Fortran compiler software is
70+successful.
71+
72+**Note**: This charm is specifically designed to run Power architecture
73+(ppc64le) systems only. It will not run on AMD (amd64) or Intel (x86_64) based
74+systems. Use `uname -m` to determine the architecture of your cloud instances.
75+
76+## Create a web repository to host the software
77+
78+To deploy this charm you need to [download][trial-xlf] the IBM XL Fortran
79+software and host it on a web server such as Apache. Run the following steps
80+with administrator access.
81+
82+Install the required packages:
83+
84+ apt-get install dpkg-dev apache2
85+
86+Create a repository on the server under the Apache repository directory:
87+
88+ cd <apache-repository-directory>
89+ mkdir ibm-xlf
90+
91+Place the downloaded IBM XL FORTRAN for Linux packages in the 'ibm-xlf'
92+directory and configure Apache server to allow access to the repository.
93+
94+Verify the IBM XL Fortran repository URL is accessible before deploying the
95+charm. You will need the URL to the directory that contains the
96+`IBM_XL_FORTRAN*.tar.gz` file to configure the ibm-xl-fortran charm.
97+
98+**Example**: `http://<server-name>:<port>/<REPOSITORY_DIR>/ibm-xlf`
99
100 Deploy
101 ------
102
103-Run the following to deploy this charm:
104-
105- juju deploy ibm-xl-fortran
106-
107-At this point IBM XL FORTRAN install will wait for you to accept the License. To install the downloaded binaries
108-you must agree to the IBM license.
109-
110-If the URL and package is configured in the config.yaml:
111-
112- juju set ibm-xl-fortran accept-ibm-xlfortran-license=True
113-
114-If not,
115-
116- juju set ibm-xl-fortran accept-ibm-xlfortran-license=True fortran-url=<repository URL> xlfortran_package_name=<Installation package name> sha_fortran=<sha sum of the xlf package>
117-
118-This charm will deploy the IBM XL FORTRAN compiler packages. For testing purpose, when you run amulet test, it will deploy the Compiler and create a sample program for compilation.
119-
120-
121+Run the following command to deploy the IBM XL Fortran charm:
122+
123+ juju deploy ibm-xl-fortran
124+
125+At this point IBM XL FORTRAN charm will wait for you to accept the license.
126+To install the downloaded binaries you must agree to the IBM license. To
127+signify you agree set the `accept-ibm-xlfortran-license` configuration option
128+to True:
129+
130+ juju set ibm-xl-fortran accept-ibm-xlfortran-license=True
131+
132+For the charm to install you also need to set URL to the package and the
133+checksum of the package to verify the software was downloaded correctly.
134+
135+These values can be set through configuration with the following command:
136+
137+ juju set ibm-xl-fortran accept-ibm-xlfortran-license=True xlfortran_url=<repository URL> xlfortran_package_name=<Installation package name> sha_fortran=<sha sum of the xlf package>
138+
139+This charm will deploy the IBM XL FORTRAN compiler packages. For testing
140+purpose, when you run amulet test, it will deploy the Compiler and create a
141+sample program for compilation.
142+
143 Configuration
144 -------------
145 See config.yaml file for more information.
146
147-`xlfortran_url`
148+`xlfortran_url`
149
150-URL to download IBM XL FORTRAN packages. The url would be in this format
151-http://<server-name>/<REPOSITORY_DIR>/ibm-xlf. If not set or set incorrectly, the charm
152-cannot install the IBM XL FORTRAN software.
153+URL to download IBM XL FORTRAN packages. The URL would be in this format
154+http://<server-name>:<port>/<REPOSITORY_DIR>/ibm-xlf. If not set or set
155+incorrectly, the charm cannot install the IBM XL FORTRAN software.
156
157 `accept-ibm-xlfortran-license`
158
159-The IBM XL FORTRAN software can only be used by accepting the terms and conditions for IBM
160-XL FORTRAN License. Setting this option to True indicates that you have read and accepted the
161-IBM XL FORTRAN terms and conditions found in the license file.
162+The IBM XL FORTRAN software can only be used by accepting the terms and
163+conditions for IBM XL FORTRAN License. Setting this option to True indicates
164+that you have read and accepted the IBM XL FORTRAN terms and conditions found
165+in the license file.
166
167 `xlfortran_package_name`
168
169@@ -90,20 +97,21 @@
170
171 `sha_fortran:`
172
173-Checksum value to check integrity of IBM XL FORTRAN package. The Charm uses
174+Checksum value to check integrity of IBM XL FORTRAN package. The charm uses
175 sha1sum to check the integrity. If empty, it does not carry out the integrity check.
176 The command to find the checksum is: "sha1sum <Package Name> | cut -d" " -f1"
177
178
179 IBM XL FORTRAN Information
180 ----------------------------
181-(1) General Information
182-Information on IBM XL FORTRAN available at the [IBM Knowledge Center] [ibm-xlf-info]
183-
184-(2) Download Information
185-Information on downloading the fully functional trial for an evaluation period of 60 days is available at [Trial Download] [trial-xlf]
186-
187-(3) Contact Information
188+#### General Information
189+Information on IBM XL FORTRAN available at the [IBM Knowledge Center][ibm-xlf-info]
190+
191+#### Download Information
192+Information on downloading the fully functional trial for an evaluation period
193+of 60 days is available at [Trial Download][trial-xlf]
194+
195+#### Contact Information
196 For issues with this charm, please contact IBM Juju Support Team <jujusupp@us.ibm.com>
197
198 <!-- Links -->
199@@ -113,6 +121,3 @@
200 [trial-xlf]: http://www.ibm.com/developerworks/downloads/r/xlfortranlinux/
201
202 [ibm-xlf-info]: http://www-01.ibm.com/support/knowledgecenter/SSAT4T_15.1.2/com.ibm.compilers.linux.doc/welcome.html
203-
204-
205-

Subscribers

People subscribed via source and target branches