Merge lp://qastaging/~graziano.obertelli/eucalyptus/2.0.0-fix-737335 into lp://qastaging/eucalyptus/2.0

Proposed by graziano obertelli
Status: Merged
Approved by: graziano obertelli
Approved revision: 1260
Merged at revision: 1259
Proposed branch: lp://qastaging/~graziano.obertelli/eucalyptus/2.0.0-fix-737335
Merge into: lp://qastaging/eucalyptus/2.0
Diff against target: 22 lines (+8/-4)
1 file modified
clc/modules/msgs/src/main/java/com/eucalyptus/images/ImageInfo.java (+8/-4)
To merge this branch: bzr merge lp://qastaging/~graziano.obertelli/eucalyptus/2.0.0-fix-737335
Reviewer Review Type Date Requested Status
graziano obertelli Approve
Review via email: mp+85715@code.qastaging.launchpad.net

Description of the change

To post a comment you must log in.
Revision history for this message
graziano obertelli (graziano.obertelli) wrote :

our QA tests identified a typo.

Compile error during compilation with javac.
/root/euca_builder/2.0.0-fix-737335/clc/modules/msgs/src/main/java/com/eucalyptus/images/ImageInfo.java:531: cannot find symbol
symbol : method getPermission()
location: class com.eucalyptus.images.ImageInfo
      if ( this.getPermission().contains( imgAuth ) ) {
               ^
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: Some input files use unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
1 error

1 error

make[1]: Leaving directory `/root/euca_builder/2.0.0-fix-737335/clc'

review: Needs Fixing
1260. By graziano obertelli

Fixes the bug for real (previous commit got a typo).

Revision history for this message
graziano obertelli (graziano.obertelli) wrote :

The new version worked well, and it is confirmed to fix the issues in our QA system.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches