Bug 977156 – Code to return

Bug 977156 - Code to return

Summary:	Code to return

Status:	CLOSED NOTABUG

Aliases:	None

Product:	Fedora
Classification:	Fedora
Component:	rpmlint (Show other bugs)
Sub Component:
Version:	18
Hardware:	Unspecified Unspecified

Priority	unspecified Severity unspecified
Target Milestone:	---
Target Release:	---
Assigned To:	Tom "spot" Callaway
QA Contact:	Fedora Extras Quality Assurance
Docs Contact:

URL:
Whiteboard:
Keywords:

Depends On:
Blocks:
	Show dependency tree / graph

Reported:	2013-06-23 18:15 EDT by MERCIER Jonathan
Modified:	2013-06-25 16:46 EDT (History)
CC List:	6 users (show)

See Also:
Fixed In Version:
Doc Type:	Bug Fix
Doc Text:
Story Points:	---
Clone Of:
Environment:
Last Closed:	2013-06-25 16:46:50 EDT
Type:	Bug
Regression:	---
Mount Type:	---
Documentation:	---
CRM:
Verified Versions:
Category:	---
oVirt Team:	---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---

Attachments	(Terms of Use)
Add an attachment (proposed patch, testcase, etc.)

Groups: None (edit)

Description MERCIER Jonathan 2013-06-23 18:15:29 EDT

why rpmdiff return 1 when it found a diff ?

The return code 1 is used for error.

a diff is not an error


this break my worklow as trap exit on error aka exit code 1

Comment 1 Todd Zullinger 2013-06-23 20:08:25 EDT

This is the standard way that diff behaves.  To quote the diff documentation¹, "An exit status of 0 means no differences were found, 1 means some differences were found, and 2 means trouble."

I don't think there is any bug in rpmdiff following the same pattern.

¹ https://www.gnu.org/software/diffutils/manual/html_node/Invoking-diff.html

Comment 2 MERCIER Jonathan 2013-06-25 16:46:50 EDT

ok no problem if that is intentional i can understand why you done this.

Thanks

Note You need to log in before you can comment on or make changes to this bug.