Bug 251180 - Build time warnings for gfs2 userland tools
Summary: Build time warnings for gfs2 userland tools
Alias: None
Product: Red Hat Enterprise Linux 5
Classification: Red Hat
Component: gfs2-utils   
(Show other bugs)
Version: 5.1
Hardware: All Linux
Target Milestone: ---
: ---
Assignee: Chris Feist
QA Contact: GFS Bugs
Depends On:
TreeView+ depends on / blocked
Reported: 2007-08-07 16:24 UTC by Robert Peterson
Modified: 2010-01-12 03:39 UTC (History)
2 users (show)

Fixed In Version: RHBA-2008-0350
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2008-05-21 17:20:01 UTC
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)
Proposed patch (7.51 KB, patch)
2007-08-27 14:32 UTC, Robert Peterson
no flags Details | Diff
Patch for HEAD branch of CVS (8.34 KB, patch)
2007-10-11 20:15 UTC, Robert Peterson
no flags Details | Diff
Patch for RHEL5 branch of CVS (12.35 KB, patch)
2007-10-11 20:18 UTC, Robert Peterson
no flags Details | Diff

External Trackers
Tracker ID Priority Status Summary Last Updated
Red Hat Product Errata RHBA-2008:0350 normal SHIPPED_LIVE gfs2-utils bug fix update 2008-05-20 12:44:56 UTC

Description Robert Peterson 2007-08-07 16:24:06 UTC
I'm cloning this bug to separate out the gfs changes from the gfs2

+++ 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)


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.


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