Bug 251180

Summary:

Build time warnings for gfs2 userland tools

Product:

Red Hat Enterprise Linux 5

Reporter:

Robert Peterson <rpeterso>

Component:

gfs2-utils

Assignee:

Chris Feist <cfeist>

Status:

CLOSED ERRATA

QA Contact:

GFS Bugs <gfs-bugs>

Severity:

medium

Docs Contact:

Priority:

medium

Version:

5.1

CC:

kanderso, rkenna

Target Milestone:

---

Target Release:

---

Hardware:

All

OS:

Linux

Whiteboard:

Fixed In Version:

RHBA-2008-0350

Doc Type:

Bug Fix

Doc Text:

Story Points:

---

Clone Of:

Environment:

Last Closed:

2008-05-21 17:20:01 UTC

Type:

---

Regression:

---

Mount Type:

---

Documentation:

---

CRM:

Verified Versions:

Category:

---

oVirt Team:

---

RHEL 7.3 requirements from Atomic Host:

Cloudforms Team:

---

Target Upstream Version:

Embargoed:

Attachments:

Description	Flags
Proposed patch	none
Patch for HEAD branch of CVS	none
Patch for RHEL5 branch of CVS	none

Description Robert Peterson 2007-08-07 16:24:06 UTC

I'm cloning this bug to separate out the gfs changes from the gfs2
changes.

+++ This bug was initially created as a clone of Bug #245573 +++

Hi Robert,

as we discussed on IRC. could you please fix the build time warning for gfs/gfs2
userland (both RHEL5 and HEAD)

Thanks
Fabio

Comment 1 Robert Peterson 2007-08-27 14:32:07 UTC

Created attachment 173601 [details]
Proposed patch

This patch eliminates the useless "pa" function and cleans up the
warnings in gfs2 userland.  Let me know if this is good enough.

Comment 2 Robert Peterson 2007-10-09 16:58:35 UTC

Requesting ack flags from the various departments.  This fix is a few
small, safe changes to the code, but I'd like to get them out of the
way before I move on to bug #325151, which is fairly serious.

Comment 3 Robert Peterson 2007-10-11 20:15:50 UTC

Created attachment 224721 [details]
Patch for HEAD branch of CVS

Here is my final patch for the HEAD branch in CVS, soon to be committed.

Comment 4 Robert Peterson 2007-10-11 20:18:24 UTC

Created attachment 224731 [details]
Patch for RHEL5 branch of CVS

This is slightly different from the HEAD patch because HEAD has 
been moving forward and therefore has differences.

Comment 5 Robert Peterson 2007-10-11 20:34:48 UTC

Code was tested on i386 on system trin-10.  Code was tested on x86_64
on system roth-01.  Patches were committed to HEAD and RHEL5 branches
for inclusion in 5.2.  Changing status to modified.

Comment 8 errata-xmlrpc 2008-05-21 17:20:01 UTC

An advisory has been issued which should help the problem
described in this bug report. This report is therefore being
closed with a resolution of ERRATA. For more information
on the solution and/or where to find the updated files,
please follow the link below. You may reopen this bug report
if the solution does not work for you.

http://rhn.redhat.com/errata/RHBA-2008-0350.html