Skip to content
Snippets Groups Projects
  1. Feb 08, 2017
  2. Jun 10, 2015
  3. Aug 19, 2014
    • Lukas Bulwahn's avatar
      making layout indentation use 4 spaces · 53796fc4
      Lukas Bulwahn authored
      The description is not indented after line breaks. All other
      information, i.e., the dependencies and SRCURI, are indented with
      four spaces after line breaks. Shell scripts are indented with
      multiples of four spaces.
      
      In recipes-ros, I located and checked for tabulators and spacing
      with these bash commands:
      
      egrep '^ [^ ]+' . --exclude *.patch -R
      egrep '^  [^ ]+' . --exclude *.patch -R
      egrep '^   [^ ]+' . --exclude *.patch -R
      egrep '^    [ ]+[^ ]+' . --exclude *.patch -R
      
      While going through the recipes and manually improving the layout,
      I also adjusted the descriptions to use maximal 100 characters
      per line.
      53796fc4
  4. Aug 18, 2014
    • Lukas Bulwahn's avatar
      improving formatting after automated style check · 8249cc8d
      Lukas Bulwahn authored
      After running oe-stylize.py on all recipes in recipes-ros, this
      commit improves the formatting of some recipes.
      
      To achieve this, I executed these commands in recipes-ros:
      find . -name *.bb | sed 's#\(.*\)#./oe-stylize.py \1 > \1_sanitized#' | sh
      find . -name *.bb | sed 's#\(.*\)#diff -Naur \1 \1_sanitized#' | sh > diffs
      
      Then, I manually inspected the diffs file, and improved the
      recipes.
      
      For the DESCRIPTION line in the addressed recipes, the line width
      of 100 characters was chosen, as the LIC_FILES_CHKSUM line and
      SRC_URI line usually are also around about 100 characters long.
      Hence, choosing a shorter line width, e.g. 80 characters, would
      have only created more line breaks, but not reduced the need to
      use a file viewer with which 100 characters line width can be
      displayed. For the github file and diff viewer and most editors
      on reasonably-sized screens, 100 character line width is no
      problem.
      8249cc8d
  5. Mar 04, 2014
  6. Sep 20, 2013
Loading