Bug 1193472

Summary: fedora-review warnings
Product: [Fedora] Fedora Reporter: Petr Lautrbach <plautrba>
Component: fedora-reviewAssignee: Stanislav Ochotnicky <sochotni>
Status: CLOSED UPSTREAM QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: leamas.alec, pingou, sochotni
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-04-16 22:39:47 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Petr Lautrbach 2015-02-17 12:33:59 UTC
Description of problem:

$ fedora-review -n policycoreutils 
INFO: Processing local files: policycoreutils
INFO: Getting .spec and .srpm Urls from : Local files in /home/plautrba/devel/git/fedora/policycoreutils/master
INFO:   --> SRPM url: file:///home/plautrba/devel/git/fedora/policycoreutils/master/policycoreutils-2.3-15.fc23.src.
INFO:   --> Spec url: file:///home/plautrba/devel/git/fedora/policycoreutils/master/policycoreutils.spec
INFO: Using review directory: /home/plautrba/devel/git/fedora/policycoreutils/master/review-policycoreutils
INFO: Using local file policycoreutils_man_ru2.tar.bz2 as Source2
INFO: Using local file system-config-selinux.png as Source3
INFO: Downloading (Source0): https://raw.githubusercontent.com/wiki/SELinuxProject/selinux/files/releases/20140506/p
INFO: Downloading (Source1): https://raw.githubusercontent.com/wiki/SELinuxProject/selinux/files/releases/20140506/s
INFO: Using local file sepolicy-icons.tgz as Source4
INFO: Running checks and generating report
INFO: Results and/or logs in: /home/plautrba/devel/git/fedora/policycoreutils/master/review-policycoreutils/results
INFO: Build completed
INFO: Installing built package(s)
INFO: Active plugins: Python, Generic, Shell-api, C/C++                                                             
INFO: Starting new HTTPS connection (1): admin.fedoraproject.org                                                    
WARNING: Illegal return from /usr/share/fedora-review/scripts/generic-large-docs.sh, code 82, output: stdout:Documentation size is 10240 bytes in 0 files.                                                                              
 stderr:./review-env.sh: line 7: unset: `PULSE_PROP_OVERRIDE_application.name': not a valid identifier              
./review-env.sh: line 7: unset: `PULSE_PROP_OVERRIDE_application.icon_name': not a valid identifier                 
                                                                                                                    
INFO:  ExclusiveArch dependency checking disabled, enable with EXARCH flag                                          
                                                                                                                    
WARNING: Illegal return from /usr/share/fedora-review/scripts/generic-excludearch.sh, code 82, output: stdout:None stderr:./review-env.sh: line 7: unset: `PULSE_PROP_OVERRIDE_application.name': not a valid identifier                
./review-env.sh: line 7: unset: `PULSE_PROP_OVERRIDE_application.icon_name': not a valid identifier                 
                                                                                                                    
WARNING: Illegal return from /usr/share/fedora-review/scripts/generic-srv-opt.sh, code 80, output: stdout:None stderr:./review-env.sh: line 7: unset: `PULSE_PROP_OVERRIDE_application.name': not a valid identifier                    
./review-env.sh: line 7: unset: `PULSE_PROP_OVERRIDE_application.icon_name': not a valid identifier                 
                                                                                                                    
WARNING: Illegal return from /usr/share/fedora-review/scripts/python-check-eggs.sh, code 80, output: stdout:None stderr:./review-env.sh: line 7: unset: `PULSE_PROP_OVERRIDE_application.name': not a valid identifier                  
./review-env.sh: line 7: unset: `PULSE_PROP_OVERRIDE_application.icon_name': not a valid identifier                 
                                                                                                                    
WARNING: Illegal return from /usr/share/fedora-review/scripts/python-egg-info.sh, code 82, output: stdout:None stderr:./review-env.sh: line 7: unset: `PULSE_PROP_OVERRIDE_application.name': not a valid identifier
./review-env.sh: line 7: unset: `PULSE_PROP_OVERRIDE_application.icon_name': not a valid identifier

WARNING: Illegal return from /usr/share/fedora-review/scripts/python-meet-guidelines.sh, code 82, output: stdout:None stderr:./review-env.sh: line 7: unset: `PULSE_PROP_OVERRIDE_application.name': not a valid identifier
./review-env.sh: line 7: unset: `PULSE_PROP_OVERRIDE_application.icon_name': not a valid identifier

WARNING: Illegal return from /usr/share/fedora-review/scripts/python-dont-download-deps.sh, code 82, output: stdout:None stderr:./review-env.sh: line 7: unset: `PULSE_PROP_OVERRIDE_application.name': not a valid identifier
./review-env.sh: line 7: unset: `PULSE_PROP_OVERRIDE_application.icon_name': not a valid identifier

WARNING: Illegal return from /usr/share/fedora-review/scripts/generic-large-data.sh, code 82, output: stdout:Arch-ed rpms have a total of 5785600 bytes in /usr/share
 stderr:./review-env.sh: line 7: unset: `PULSE_PROP_OVERRIDE_application.name': not a valid identifier
./review-env.sh: line 7: unset: `PULSE_PROP_OVERRIDE_application.icon_name': not a valid identifier

Review template in: /home/plautrba/devel/git/fedora/policycoreutils/master/review-policycoreutils/review.txt
fedora-review is automated tool, but *YOU* are responsible for manually
reviewing the results and finishing the review. Do not just copy-paste
the results without understanding them.


Version-Release number of selected component (if applicable):
fedora-review-0.5.2-2.fc22.noarch

Comment 1 Alec Leamas 2015-04-16 22:39:47 UTC
This is fixed in 
https://fedorahosted.org/FedoraReview/changeset/18d98aa3d0f441f7c