From 95b154183516d6ba8bf9367c20fbadddee680391 Mon Sep 17 00:00:00 2001 From: Karen Arutyunov Date: Fri, 17 Aug 2018 23:24:59 +0300 Subject: Add submit-dir --- brep/submit/buildfile | 6 +- brep/submit/submit-dir.in | 97 +++++++++++++++++++++++++ brep/submit/submit-git.in | 14 ++++ brep/submit/submit.bash.in | 114 +++++++++++++++++++++++++++++ brep/submit/submit.in | 176 --------------------------------------------- 5 files changed, 230 insertions(+), 177 deletions(-) create mode 100644 brep/submit/submit-dir.in create mode 100644 brep/submit/submit-git.in create mode 100644 brep/submit/submit.bash.in delete mode 100644 brep/submit/submit.in (limited to 'brep') diff --git a/brep/submit/buildfile b/brep/submit/buildfile index 48cf676..f25b6e3 100644 --- a/brep/submit/buildfile +++ b/brep/submit/buildfile @@ -5,4 +5,8 @@ import mods = libbutl.bash%bash{manifest-parser} import mods += libbutl.bash%bash{manifest-serializer} -exe{brep-submit}: in{submit} $mods +exe{brep-submit-dir}: in{submit-dir} bash{submit} $mods +bash{submit}: in{submit} $mods # @@ Currently doesn't depend on manifest-parser. + +#install.bin.subdirs = true +#install.subdirs = true diff --git a/brep/submit/submit-dir.in b/brep/submit/submit-dir.in new file mode 100644 index 0000000..ce8f134 --- /dev/null +++ b/brep/submit/submit-dir.in @@ -0,0 +1,97 @@ +#!/usr/bin/env bash + +# file : brep/submit/submit-dir.in +# copyright : Copyright (c) 2014-2018 Code Synthesis Ltd +# license : MIT; see accompanying LICENSE file + +# Package submission handler example. +# +# Validate the package archive located in the specified submission directory +# extracting and parsing the package manifest. Remove the submission directory +# if simulating. Write the submission result manifest to stdout. +# +usage="usage: $0 " + +# Diagnostics. +# +verbose=true + +trap "{ exit 1; }" ERR +set -o errtrace # Trap ERR in functions. + +@import libbutl/manifest-parser@ +@import libbutl/manifest-serializer@ + +@import brep/submit/submit@ + +# Parse the submission request manifest and obtain the archive path as well +# as the simulate value. +# +trace "parsing $dir/request.manifest" +butl_manifest_parser_start "$dir/request.manifest" + +archive= +simulate= + +while IFS=: read -ru "$butl_manifest_parser_ofd" -d '' n v; do + case "$n" in + archive) archive="$v" ;; + simulate) simulate="$v" ;; + esac +done + +butl_manifest_parser_finish + +if [ -z "$archive" ]; then + error "archive manifest value expected" +fi + +if [ -n "$simulate" -a "$simulate" != "success" ]; then + trace "unrecognized simulation outcome '$simulate'" + result_manifest 400 "unrecognized simulation outcome" + exit 0 +fi + +manifest="$dir/package.manifest" + +extract_package_manifest "$dir/$archive" "$manifest" + +# Parse the package manifest and obtain the package name and version. +# +trace "parsing $manifest" +butl_manifest_parser_start "$manifest" + +name= +version= +project= + +while IFS=: read -ru "$butl_manifest_parser_ofd" -d '' n v; do + case "$n" in + name) name="$v" ;; + version) version="$v" ;; + project) project="$v" ;; + esac +done + +butl_manifest_parser_finish + +if [ -z "$name" ]; then + error "name manifest values expected" +fi + +if [ -z "$version" ]; then + error "version manifest values expected" +fi + +if [ -z "$project" ]; then + project="$name" +fi + +if [ -n "$simulate" ]; then + rm -r "$dir" + trace "$name/$version submission is simulated" +else + trace "$name/$version submission is queued" +fi + +result_manifest 200 "$name/$version submission is queued" "$reference" diff --git a/brep/submit/submit-git.in b/brep/submit/submit-git.in new file mode 100644 index 0000000..0238c4e --- /dev/null +++ b/brep/submit/submit-git.in @@ -0,0 +1,14 @@ +#!/usr/bin/env bash + +# file : brep/submit/submit-git.in +# copyright : Copyright (c) 2014-2018 Code Synthesis Ltd +# license : MIT; see accompanying LICENSE file + +# Package submission handler. +# +# @@ TODO doc/spec. +# +usage="usage: $0 " + +trap "{ exit 1; }" ERR +set -o errtrace # Trap ERR in functions. diff --git a/brep/submit/submit.bash.in b/brep/submit/submit.bash.in new file mode 100644 index 0000000..d6f0258 --- /dev/null +++ b/brep/submit/submit.bash.in @@ -0,0 +1,114 @@ +# file : brep/submit/submit.bash.in +# copyright : Copyright (c) 2014-2018 Code Synthesis Ltd +# license : MIT; see accompanying LICENSE file + +# @@ Should we make this module besides defining utility functions to also +# parse and remove from args the common arguments (directory)? +# + +if [ "$brep_submit" ]; then + return 0 +else + brep_submit=true +fi + +@import libbutl/manifest-serializer@ + +# Diagnostics. +# +self="$(basename $0)" + +# Result reference (assigned later). +# +reference= + +# Normally the brep module's log record looks like this: +# +# [Mon Jul 23 17:48:46.945079 2018] [brep:error] [pid 123:tid 456] [brep::submit::init]: error description +# +# We will use the (almost) same format for our diagnostics (redirected to the +# Apache's error_log) so it can easily be attributed to the brep module. +# +function info () # +{ + local severity="$1" + shift + + # Note: %N is Linux-specific. + # + local ts + if ! ts="$(date +"%a %b %d %H:%M:%S.%6N %Y")"; then + ts= + fi + + echo "[$ts] [brep:$severity] [ref $reference] [$self]: $*" 1>&2; +} + +function error () { info "error" "$*"; exit 1; } +function trace () { if [ "$verbose" ]; then info "info" "$*"; fi } + +# Submission data directory. +# +# @@ Doesn't comply with implementation assuming that directory comes before +# submit-handler-arguments. Can we change the doc? +# +dir="${1%/}" # @@ Call it directory? +shift + +if [ -z "$dir" ]; then + error "$usage" +fi + +if [ ! -d "$dir" ]; then + error "'$dir' does not exist or is not a directory" +fi + +reference="$(basename $dir)" + +# Serialize one manifest name/value pair. +# +function serialize () # +{ + printf "%s:%s\0" "$1" "$2" >&"$butl_manifest_serializer_ifd" +} + +# Serialize the submission result manifest to stdout. +# +function result_manifest () # [] +{ + local sts="$1" + local msg="$2" + local ref="$3" + + trace "serializing result manifest" + butl_manifest_serializer_start + + serialize "" "1" # Start of manifest. + serialize "status" "$sts" + serialize "message" "$msg" + + if [ -n "$ref" ]; then + serialize "reference" "$ref" + fi + + butl_manifest_serializer_finish +} + +# Verify the archive is a valid bpkg package and extract its manifest file. +# +function extract_package_manifest () # +{ + local arc="$1" + local man="$2" + + # Should we remove the submission directory with an invalid package? + # Probably it's better to leave it for potential investigation. Note that we + # can always grep for such directories based on the result.manifest file + # they contain. + # + if ! bpkg pkg-verify --manifest "$arc" >"$man" 2>/dev/null; then + trace "$arc is not a valid package" + result_manifest 400 "archive is not a valid package (run bpkg pkg-verify for details)" + exit 0 + fi +} diff --git a/brep/submit/submit.in b/brep/submit/submit.in deleted file mode 100644 index c4ef08b..0000000 --- a/brep/submit/submit.in +++ /dev/null @@ -1,176 +0,0 @@ -#!/usr/bin/env bash - -# file : brep/submit/submit.in -# copyright : Copyright (c) 2014-2018 Code Synthesis Ltd -# license : MIT; see accompanying LICENSE file - -# Package submission handler example. -# -# Validate the package archive located in the specified submission directory -# extracting and parsing the package manifest. Remove the submission directory -# if simulating. Write the submission result manifest to stdout. -# -usage="usage: $0 " - -trap "{ exit 1; }" ERR -set -o errtrace # Trap ERR in functions. - -@import libbutl/manifest-parser@ -@import libbutl/manifest-serializer@ - -# Diagnostics. -# -self="$(basename $0)" -verbose= # true - -# Result reference (assigned later). -# -reference= - -# Normally the brep module's log record looks like this: -# -# [Mon Jul 23 17:48:46.945079 2018] [brep:error] [pid 123:tid 456] [brep::submit::init]: error description -# -# We will use the (almost) same format for our diagnostics (redirected to the -# Apache's error_log) so it can easily be attributed to the brep module. -# -function info () # -{ - local severity="$1" - shift - - # Note: %N is Linux-specific. - # - local ts - if ! ts="$(date +"%a %b %d %H:%M:%S.%6N %Y")"; then - ts= - fi - - echo "[$ts] [brep:$severity] [ref $reference] [$self]: $*" 1>&2; -} - -function error () { info "error" "$*"; exit 1; } -function trace () { if [ "$verbose" ]; then info "info" "$*"; fi } - -dir="${1%/}" - -if [ -z "$dir" ]; then - error "$usage" -fi - -if [ ! -d "$dir" ]; then - fail "'$dir' does not exist or is not a directory" -fi - -reference="$(basename $dir)" - -# Parse the submission request manifest and obtain the archive path as well -# as the simulate value. -# -trace "parsing $dir/request.manifest" -butl_manifest_parser_start "$dir/request.manifest" - -archive= -simulate= - -while IFS=: read -ru "$butl_manifest_parser_ofd" -d '' n v; do - case "$n" in - archive) archive="$v" ;; - simulate) simulate="$v" ;; - esac -done - -butl_manifest_parser_finish - -if [ -z "$archive" ]; then - error "archive manifest value expected" -fi - -# Serialize one manifest name/value pair. -# -function serialize () # -{ - printf "%s:%s\0" "$1" "$2" >&"$butl_manifest_serializer_ifd" -} - -# Serialize the submission result manifest to stdout. -# -function result_manifest () # [] -{ - local sts="$1" - local msg="$2" - local ref="$3" - - trace "serializing result manifest" - butl_manifest_serializer_start - - serialize "" "1" # Start of manifest. - serialize "status" "$sts" - serialize "message" "$msg" - - if [ -n "$ref" ]; then - serialize "reference" "$ref" - fi - - butl_manifest_serializer_finish -} - -if [ -n "$simulate" -a "$simulate" != "success" ]; then - trace "unrecognized simulation outcome '$simulate'" - result_manifest 400 "unrecognized simulation outcome" - exit 0 -fi - -# Verify the archive is a valid bpkg package and extract its manifest file. -# -# Should we remove the submission directory with an invalid package? Probably -# it's better to leave it for potential investigation. Note that we can always -# grep for such directories based on the result.manifest file they contain. -# -manifest="$dir/package.manifest" - -if ! bpkg pkg-verify --manifest "$dir/$archive" >"$manifest" 2>/dev/null; then - trace "$dir/$archive is not a valid package" - result_manifest 400 "archive is not a valid package (run bpkg pkg-verify for details)" - exit 0 -fi - -# Parse the package manifest and obtain the package name and version. -# -trace "parsing $manifest" -butl_manifest_parser_start "$manifest" - -name= -version= -project= - -while IFS=: read -ru "$butl_manifest_parser_ofd" -d '' n v; do - case "$n" in - name) name="$v" ;; - version) version="$v" ;; - project) project="$v" ;; - esac -done - -butl_manifest_parser_finish - -if [ -z "$name" ]; then - error "name manifest values expected" -fi - -if [ -z "$version" ]; then - error "version manifest values expected" -fi - -if [ -z "$project" ]; then - project="$name" -fi - -if [ -n "$simulate" ]; then - rm -r "$dir" - trace "$name/$version submission is simulated" -else - trace "$name/$version submission is queued" -fi - -result_manifest 200 "$name/$version submission is queued" "$reference" -- cgit v1.1