Bug 818884 - Typo in logs of graph.y and y.tab.c
Summary: Typo in logs of graph.y and y.tab.c
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: logging
Version: mainline
Hardware: Unspecified
OS: Unspecified
low
medium
Target Milestone: ---
Assignee: Amar Tumballi
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks: 817967
TreeView+ depends on / blocked
 
Reported: 2012-05-04 09:25 UTC by Anush Shetty
Modified: 2013-12-19 00:08 UTC (History)
2 users (show)

Fixed In Version: glusterfs-3.4.0
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-07-24 17:26:29 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

Description Anush Shetty 2012-05-04 09:25:53 UTC
Description of problem: Temparory should be temporary.

In libglusterfs/src/graph.y and libglusterfs/src/y.tab.c 

if (tmp_file == NULL) {
                gf_log ("parser", GF_LOG_ERROR,
                        "cannot create temparory file");

                glusterfs_graph_destroy (graph);
                return NULL;
        }




Version-Release number of selected component (if applicable): Upstream


How reproducible: N/A

  
Actual results:

[2012-05-04 08:20:09.368261] E [graph.y:577:glusterfs_graph_construct] 0-parser: cannot create temparory file




Expected results:


Additional info:

Comment 1 Anand Avati 2012-05-05 19:41:05 UTC
CHANGE: http://review.gluster.com/3273 (Fix spelling error) merged in release-3.2 by Anand Avati (avati)

Comment 2 Amar Tumballi 2012-05-07 08:46:10 UTC
Applicable on master too. Joe, will you be sending a patch for this? or should I ask one of the dev to take care of this for master?

Comment 3 Anand Avati 2012-05-15 09:03:41 UTC
CHANGE: http://review.gluster.com/3331 (Fix spelling error) merged in master by Vijay Bellur (vijay)

Comment 4 Anush Shetty 2012-05-18 13:11:21 UTC
Fixed in master 19394ef5f133d18aea757129bd903307814f1afb

Comment 5 Vijay Bellur 2013-03-13 03:06:58 UTC
CHANGE: http://review.gluster.org/4661 (Fix spelling error) merged in release-3.3 by Anand Avati (avati)


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