diff options
author | Ville Skyttä <ville.skytta@iki.fi> | 2009-04-09 20:35:42 +0300 |
---|---|---|
committer | Ville Skyttä <ville.skytta@iki.fi> | 2009-04-09 20:35:42 +0300 |
commit | 886587d4a06a67f419dc0cbf10e13d0f546dc476 (patch) | |
tree | d89754d89e7d74c0953ba5a4e5580ff32bc403b1 | |
parent | b47af0bde091a3184b3e70b8b1fa44a27da2967d (diff) | |
download | fedora-packager-886587d4a06a67f419dc0cbf10e13d0f546dc476.tar.gz fedora-packager-886587d4a06a67f419dc0cbf10e13d0f546dc476.tar.xz fedora-packager-886587d4a06a67f419dc0cbf10e13d0f546dc476.zip |
Improve package lookup error control.
-rwxr-xr-x | src/fedoradev-pkgowners | 14 |
1 files changed, 10 insertions, 4 deletions
diff --git a/src/fedoradev-pkgowners b/src/fedoradev-pkgowners index 1b50daf..ada5753 100755 --- a/src/fedoradev-pkgowners +++ b/src/fedoradev-pkgowners @@ -110,13 +110,19 @@ this_init() lookup_package() { local package_info="$(grep "${filter_for}|${1}|" "${file_pkgdbfiledata}")" - local owner="$(echo ${package_info} | cut -d '|' -f 4)" - if [[ ! "${owner}" ]]; then - # grep did not find anything or something else went wrong - echo "Could not find owner for ${1}" >&2 + if [[ ! "${package_info}" ]]; then + # grep did not find anything or something else went wrong, eg. + # https://fedorahosted.org/packagedb/ticket/138 + echo "Warning: could not find info for ${1}" >&2 return 1 fi + # owner? + local owner="$(echo ${package_info} | cut -d '|' -f 4)" + if [[ ! "${owner}" || "${owner}" == orphan ]]; then + # No owner, assume orphan + owner="(orphan)" + fi # co-maintainers? if [[ "${print_comaintainers}" ]]; then |