From ca9ff17d273f1d02abd34ee0913a87f93397cd11 Mon Sep 17 00:00:00 2001 From: Stephen Finucane Date: Wed, 4 Oct 2017 11:41:57 +0100 Subject: [PATCH] Remove "cleanup" tools There were a couple of scripts here: - prettify.py - remove_trailing_whitespaces.sh - remove_unnecessary_spaces.py - retf Most of them are self-explanatory with the exception of 'retf', which I don't understand but which looks untouched in years and therefore useless. For the other three, we have tooling in place to ensure these "issues" don't get into source control and, even if they did, there's questionable value in rewriting stuff just to remove excess whitespace. Kill them all. Change-Id: Ifbd2d50a669251484056552da21d23906f8d6178 --- cleanup/prettify.py | 72 ---- cleanup/remove_trailing_whitespaces.sh | 24 -- cleanup/remove_unnecessary_spaces.py | 69 ---- cleanup/retf/README.md | 69 ---- cleanup/retf/disabled_rules.yaml | 1 - cleanup/retf/retf.py | 307 ------------------ ...at-doc-tools-cleanup-1a79e2c200232489.yaml | 11 + setup.cfg | 1 - tox.ini | 5 +- 9 files changed, 13 insertions(+), 546 deletions(-) delete mode 100755 cleanup/prettify.py delete mode 100755 cleanup/remove_trailing_whitespaces.sh delete mode 100755 cleanup/remove_unnecessary_spaces.py delete mode 100644 cleanup/retf/README.md delete mode 100644 cleanup/retf/disabled_rules.yaml delete mode 100755 cleanup/retf/retf.py diff --git a/cleanup/prettify.py b/cleanup/prettify.py deleted file mode 100755 index 78936900..00000000 --- a/cleanup/prettify.py +++ /dev/null @@ -1,72 +0,0 @@ -#!/usr/bin/env python - -"""A script to prettify HTML and XML syntax. - -Some examples of the prettified syntax are available -in the following changes: - -* https://review.openstack.org/#/c/98652/ -* https://review.openstack.org/#/c/98653/ -* https://review.openstack.org/#/c/98655/ -""" - -# Licensed under the Apache License, Version 2.0 (the "License"); you may -# not use this file except in compliance with the License. You may obtain -# a copy of the License at -# -# http://www.apache.org/licenses/LICENSE-2.0 -# -# Unless required by applicable law or agreed to in writing, software -# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT -# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the -# License for the specific language governing permissions and limitations -# under the License. - - -from __future__ import print_function -import argparse -import sys - -from bs4 import BeautifulSoup - - -def parse_command_line_arguments(): - """Parse the command line arguments.""" - parser = argparse.ArgumentParser() - parser.add_argument("--write-changes", action="store_true", default=False, - help="Write prettified XML or HTML syntax " - "back to file.") - parser.add_argument("file", type=str, default=None, - help="A XML or HTML File to prettify.") - return parser.parse_args() - - -def main(): - """Entry point for this script.""" - - args = parse_command_line_arguments() - - try: - soup = BeautifulSoup(open(args.file)) - except IOError as exception: - print("ERROR: File '%s' could not be parsed: %s" - % (args.file, exception)) - return 1 - - if args.write_changes: - try: - with open(args.file, 'wb') as output: - prettified = soup.prettify(encoding="utf8") - output.write(prettified) - except IOError as exception: - print("ERROR: File '%s' could not be written: %s" - % (args.file, exception)) - return 1 - else: - prettified = soup.prettify(encoding="utf8") - print(prettified) - - return 0 - -if __name__ == '__main__': - sys.exit(main()) diff --git a/cleanup/remove_trailing_whitespaces.sh b/cleanup/remove_trailing_whitespaces.sh deleted file mode 100755 index 7dd7163f..00000000 --- a/cleanup/remove_trailing_whitespaces.sh +++ /dev/null @@ -1,24 +0,0 @@ -#!/bin/sh - -## copyright: B1 Systems GmbH , 2013. -## author: Christian Berendt , 2013. - -# Licensed under the Apache License, Version 2.0 (the "License"); you may -# not use this file except in compliance with the License. You may obtain -# a copy of the License at -# -# http://www.apache.org/licenses/LICENSE-2.0 -# -# Unless required by applicable law or agreed to in writing, software -# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT -# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the -# License for the specific language governing permissions and limitations -# under the License. - -# Call ./tools/cleanup/remove_trailing_whitespaces.sh in the -# root of openstack-manuals. - -files=$(find doc -name *.xml -not -name pom.xml) -for file in $files; do - sed -i -e 's/[[:space:]]*$//' $file -done diff --git a/cleanup/remove_unnecessary_spaces.py b/cleanup/remove_unnecessary_spaces.py deleted file mode 100755 index bf175fca..00000000 --- a/cleanup/remove_unnecessary_spaces.py +++ /dev/null @@ -1,69 +0,0 @@ -#!/usr/bin/env python - -# copyright: B1 Systems GmbH , 2013. - -# Licensed under the Apache License, Version 2.0 (the "License"); you may -# not use this file except in compliance with the License. You may obtain -# a copy of the License at -# -# http://www.apache.org/licenses/LICENSE-2.0 -# -# Unless required by applicable law or agreed to in writing, software -# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT -# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the -# License for the specific language governing permissions and limitations -# under the License. - -# Call ./tools/cleanup/remove_unnecessary_spaces.py in the -# root of openstack-manuals. - -import os -import re -import shutil -import tempfile - -# should be the same like in tools/validate.py -FILE_EXCEPTIONS = ['ha-guide-docinfo.xml', - 'bk001-ch003-associate-general.xml'] - -elements = [ - 'listitem', - 'para', - 'td', - 'th', - 'command', - 'literal', - 'title', - 'caption', - 'filename', - 'userinput', - 'programlisting' -] - -checks = [] -for element in elements: - checks.append(re.compile("(.*<%s>)\s+([\w\-().:!?{}\[\]]+.*\n)" - % element)), - checks.append(re.compile("(.*[\w\-().:!?{}\[\]]+)\s+(<\/%s>.*\n)" - % element)) - -for root, dirs, files in os.walk('doc/'): - for f in files: - if (not (f.endswith('.xml') and - f != 'pom.xml' and - f not in FILE_EXCEPTIONS)): - continue - docfile = os.path.abspath(os.path.join(root, f)) - tmpfile = tempfile.mkstemp() - tmpfd = os.fdopen(tmpfile[0], "w") - match = False - for line in open(docfile, 'r'): - for check in checks: - if check.match(line): - line = check.sub(r"\1\2", line) - match = True - tmpfd.write(line) - tmpfd.close() - if match: - shutil.copyfile(tmpfile[1], docfile) - os.unlink(tmpfile[1]) diff --git a/cleanup/retf/README.md b/cleanup/retf/README.md deleted file mode 100644 index ae8e03dc..00000000 --- a/cleanup/retf/README.md +++ /dev/null @@ -1,69 +0,0 @@ -# retf.py - -This script applies a set of regular expressions onto a set of files -to automatically identify and fix typographical errors. - -## What does RETF mean? - -RETF means RegExTypoFix or Regular Expression Typographical error Fixer -and is a set of regular expressions to find and fix common misspellings -and grammatical errors. - -The regular expressions are available at -https://en.wikipedia.org/wiki/Wikipedia:AutoWikiBrowser/Typos. - -## Usage - -There are two ways to define the set of files. First you can simply add -single files using the parameter ```--file```. - -```$ ./retf.py --file path/to/file1 path/to/file2 path/to/file3``` - -Also you can specify paths using the parameter ```--path``` that should be -scanned for files. - -```$ ./retf.py --path path/with/files/1 path/with/files/2``` - -To not use all files inside the specified paths it's possible to filter -by the file extension. - -```$ ./retf.py --path path/with/files --extension xml txt rst``` - -It's possible to use the parameters ```--path``` and ```--file``` together. - -By default the script will only check for findings in all specified files. - -To automatically write back resolved findings add the parameter -```--write-changes```. Findings will then be written to a copy with -the ending ```.retf```. - -To fix findings directly in the files add the parameter -```--in-place```. Findings will than be fixed directly in the files. A backup file -with the ending ```.orig``` will be created. To disable backups add the -paramter ```--no-backup```. - -To only check if there are findings inside the defined set of files add - -To download the latest RETF rules from Wikipedia use the parameter ```--download```. - -## Needed Python modules - -* beautifulsoup4 / bs4 (https://pypi.python.org/pypi/beautifulsoup4) -* glob2 (https://pypi.python.org/pypi/glob2) -* pyyaml (https://pypi.python.org/pypi/pyaml) -* regex (https://pypi.python.org/pypi/regex) -* six (https://pypi.python.org/pypi/six) - -To install the needed modules you can use pip or the package management system included -in your distribution. When using the package management system maybe the name of the -packages differ. When using pip it's maybe necessary to install some development packages. -For example on Ubuntu 14.04 LTS you have to install ```libyaml-dev``` for ```pyyaml``` -and ```python-dev``` for ```regex```. - -``` -$ pip install beautifulsoup4 -$ pip install glob2 -$ pip install pyyaml -$ pip install regex -$ pip install six -``` diff --git a/cleanup/retf/disabled_rules.yaml b/cleanup/retf/disabled_rules.yaml deleted file mode 100644 index ed97d539..00000000 --- a/cleanup/retf/disabled_rules.yaml +++ /dev/null @@ -1 +0,0 @@ ---- diff --git a/cleanup/retf/retf.py b/cleanup/retf/retf.py deleted file mode 100755 index 07990ec6..00000000 --- a/cleanup/retf/retf.py +++ /dev/null @@ -1,307 +0,0 @@ -#!/usr/bin/env python - -"""This script applies a set of regular expressions onto a set of files -to automatically identify and fix typographical errors. -""" - -# Licensed under the Apache License, Version 2.0 (the "License"); you may -# not use this file except in compliance with the License. You may obtain -# a copy of the License at -# -# http://www.apache.org/licenses/LICENSE-2.0 -# -# Unless required by applicable law or agreed to in writing, software -# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT -# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the -# License for the specific language governing permissions and limitations -# under the License. - -# Based on the idea of 'Topy' written by Marti Raudsepp . -# Topy is available on Github at https://github.com/intgr/topy. - -import argparse -import logging -import os -import shutil -import sys - -from six.moves.urllib import error as urlerr -from six.moves.urllib import request as urlreq - -from bs4 import BeautifulSoup -import glob2 -import regex -import six -import yaml - - -class DownloadRetfListingFailed(Exception): - """Exception for failed downloads of the RETF listing. - - Exception will be raised when the download of the RETF - listing failed or the destination file could not be written. - - """ - - pass - - -def download_listing(dest): - """Download the latest RETF listing from Wikipedia.""" - logger = logging.getLogger('retf') - try: - url = ('https://en.wikipedia.org/wiki/Wikipedia:AutoWikiBrowser/' - 'Typos?action=raw') - logger.debug("Downloading latest RETF listing from %s into %s.", - url, dest) - response = urlreq.urlopen(url) - data = response.read() - logger.info("Downloading latest RETF listing from %s succeeded.", url) - except urlerr.HTTPError as ex: - raise DownloadRetfListingFailed(six.text_type(ex)) - except urlerr.URLError as ex: - raise DownloadRetfListingFailed(six.text_type(ex)) - - try: - with open(dest, 'w+') as write: - write.write(data) - logger.info("Writing RETF listing to file %s succeeded.", dest) - except IOError as ex: - raise DownloadRetfListingFailed(six.text_type(ex)) - - -def soupify_listing(src): - """Parse a RETF listing.""" - return BeautifulSoup(open(src)) - - -def generate_listing(src): - """Compile all regular expressions in a RETF listing.""" - logger = logging.getLogger('retf') - result = [] - - soup = soupify_listing(src) - - for typo in soup.findAll('typo'): - try: - word = typo.attrs.get('word').encode('utf8') - find = typo.attrs.get('find').encode('utf8') - replace = typo.attrs.get('replace').encode('utf8') - replace = replace.replace(b'$', b'\\') - except AttributeError: - continue - - # pylint: disable=W0703 - try: - logger.debug("Compiling regular expression: %s.", find) - compiled = regex.compile(find, flags=regex.V1) - except Exception: - logger.error("Compilation of regular expression %f failed.", find) - continue - # pylint: enable=W0703 - - entry = { - 'description': word, - 'find': find, - 'replace': replace, - 'regex': compiled - } - - result.append(entry) - - logger.debug("Compiled %d regular expression(s).", len(result)) - - return result - - -def load_text_from_file(src): - """Load content from a file.""" - logger = logging.getLogger('retf') - logger.debug("Loading text from file %s.", src) - with open(src, 'rb') as fpointer: - text = fpointer.read() - - return text - - -def write_text_to_file(dest, text, no_backup, in_place): - """Write content into a file.""" - logger = logging.getLogger('retf') - - if not no_backup: - logger.debug("Copying %s to backup file %s.orig.", dest, dest) - shutil.copy2(dest, "%s.orig" % dest) - - if not in_place: - dest = "%s.retf" % dest - - logger.debug("Writing text to file %s.", dest) - with open(dest, 'wb') as fpointer: - fpointer.write(text) - - -def initialize_logging(debug, less_verbose): - """Initialize the Logger.""" - logger = logging.getLogger(name='retf') - formatter = logging.Formatter('%(asctime)s %(levelname)-8s %(message)s') - handler = logging.StreamHandler() - handler.setFormatter(formatter) - logger.addHandler(handler) - - logger.setLevel(logging.INFO) - - if less_verbose: - logger.setLevel(logging.WARN) - - if debug: - logger.setLevel(logging.DEBUG) - - return logging.getLogger('retf') - - -def parse_command_line_arguments(): - """Parse the command line arguments.""" - parser = argparse.ArgumentParser() - parser.add_argument("--debug", help="Print debugging messages.", - action="store_true", default=False) - parser.add_argument("--download", help="Download the latest RETF listing.", - action="store_true", default=False) - parser.add_argument("--less-verbose", help="Be less verbose.", - action="store_true", default=False) - parser.add_argument("--no-backup", help="Don't backup files.", - action="store_true", default=False) - parser.add_argument("--in-place", help="Resolve found errors in place.", - action="store_true", default=False) - parser.add_argument("--write-changes", action="store_true", default=False, - help="Write resolved findings back to files.") - parser.add_argument("--disabled", type=str, default=None, - help="File containing the disabled rules.") - parser.add_argument("--listing", help="File containing the RETF listing.", - type=str, default=os.path.join( - os.path.dirname(os.path.realpath(__file__)), - 'retf.lst')) - parser.add_argument("--path", type=str, nargs='*', default=[], - help="Path(s) that should be checked.") - parser.add_argument("--extension", type=str, nargs='*', default=[], - help="Only check files with specified extension(s).") - parser.add_argument("--file", nargs='*', type=str, default=[], - help="File(s) to check for typographical errors.") - return (parser, parser.parse_args()) - - -def load_disabled_rules(src): - """Load disabled rules from YAML file.""" - logger = logging.getLogger('retf') - listing = [] - - if src: - try: - listing = yaml.safe_load(open(src)) - for rule in listing: - logger.debug("Rule '%s' is disabled.", rule) - - except IOError: - logger.error("loading disabled rules from file %s failed", src) - - return listing - - -def get_file_listing(paths, files, extensions): - """Generate listing with all files that should be check.""" - result = [] - if files: - result += files - - # pylint: disable=E1101 - for path in paths: - if extensions: - for extension in extensions: - result += glob2.glob("%s/**/*.%s" % (path, extension)) - else: - result += glob2.glob("%s/**/*" % path) - # pylint: enable=E1101 - - return result - - -def check_file(src, rules, disabled): - """Applies a set of rules on a file.""" - logger = logging.getLogger('retf') - logger.info("Checking file %s for typographical errors.", src) - content = load_text_from_file(src) - findings = 0 - - for rule in rules: - if rule.get('description') in disabled: - continue - - logger.debug("%s: checking rule '%s'.", src, - rule.get('description')) - logger.debug(rule.get('find')) - newcontent, count = rule.get('regex').subn( - rule.get('replace'), content - ) - - if count > 0: - logger.warning("%d match(s) in file %s : %s.", count, src, - rule.get('description')) - findings += count - content = newcontent - - return (findings, content) - - -def main(): - """Entry point for this script.""" - - parser, args = parse_command_line_arguments() - logger = initialize_logging(args.debug, args.less_verbose) - - result = 0 - - if args.download: - try: - download_listing(args.listing) - except DownloadRetfListingFailed as ex: - logger.error("Downloading latest RETF listing failed: %s.", ex) - result = 1 - - if not args.path and not args.file and not args.download: - parser.print_help() - result = 2 - - if not result and not os.path.isfile(args.listing): - logger.error("RETF listing not found at %s.", args.listing) - logger.info("Please download the RETF listing first by using the " - "parameter --download.") - result = 1 - - if not result: - files = get_file_listing(args.path, args.file, args.extension) - - rules = generate_listing(args.listing) - disabled = load_disabled_rules(args.disabled) - - all_findings = 0 - for check in files: - if not os.path.isfile(check): - continue - - (findings, content) = check_file(check, rules, disabled) - - if findings > 0: - all_findings += findings - logger.warning("%s finding(s) in file %s.", findings, check) - - if findings > 0 and args.write_changes: - write_text_to_file(check, content, args.no_backup, - args.in_place) - - if all_findings > 0: - logger.warning("%s finding(s) in all checked files.", all_findings) - result = 1 - - return result - -if __name__ == "__main__": - sys.exit(main()) diff --git a/releasenotes/notes/the-great-doc-tools-cleanup-1a79e2c200232489.yaml b/releasenotes/notes/the-great-doc-tools-cleanup-1a79e2c200232489.yaml index 9ebb70ea..d05a6759 100644 --- a/releasenotes/notes/the-great-doc-tools-cleanup-1a79e2c200232489.yaml +++ b/releasenotes/notes/the-great-doc-tools-cleanup-1a79e2c200232489.yaml @@ -12,3 +12,14 @@ upgrade: superseded by the ``cliff.sphinxext`` `Sphinx extensions`__. __ https://docs.openstack.org/cliff/latest/user/sphinxext.html + - | + The following scripts and tools have been removed: + + - prettify.py$ + - remove_trailing_whitespaces.sh$ + - remove_unnecessary_spaces.py$ + - retf$ + + These were all indented to cleanup docs once they were already in tree, but + our tooling and review process should be sufficient to catch most issues + and there's limited value in submitting whitespace-fixup patches. diff --git a/setup.cfg b/setup.cfg index be5e6825..bb801b6d 100644 --- a/setup.cfg +++ b/setup.cfg @@ -24,7 +24,6 @@ packages = os_doc_tools data_files = share/openstack-doc-tools/sitemap = sitemap/* - share/openstack-doc-tools/cleanup = cleanup/* scripts = bin/doc-tools-check-languages bin/doc-tools-build-rst diff --git a/tox.ini b/tox.ini index 13168756..28de325b 100644 --- a/tox.ini +++ b/tox.ini @@ -25,11 +25,10 @@ commands = doc8 -e txt -e rst doc/source/ HACKING.rst # Run bashate during pep8 runs to ensure violations are caught by # the check and gate queues. - bashate bin/doc-tools-check-languages \ - cleanup/remove_trailing_whitespaces.sh + bashate bin/doc-tools-check-languages [testenv:pylint] -commands = pylint os_doc_tools cleanup sitemap +commands = pylint os_doc_tools sitemap [testenv:releasenotes] commands = sphinx-build -a -E -W -d releasenotes/build/doctrees -b html releasenotes/source releasenotes/build/html