View Issue Details Jump to Notes ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0014188CMakeCMakepublic2013-06-02 18:242013-11-04 09:33
ReporterYichao Yu 
Assigned ToBrad King 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformLinuxOSArch LinuxOS Version
Product Version 
Target VersionCMake 2.8.12Fixed in VersionCMake 2.8.12 
Summary0014188: variable_watch print REMOVED_ACCESS instead of MODIFIED_ACCESS when set the value of the variable.
DescriptionSee summary and steps to reproduce.
Steps To Reproduceset(b "a")
variable_watch(b)
set(b "b")

expected output:
* Variable "b" was accessed using MODIFIED_ACCESS in: ....

actual output:
* Variable "b" was accessed using REMOVED_ACCESS in: ....
TagsNo tags attached.
Attached Filespatch file icon 0001-BUG-0014188-variable_watch-print-REMOVED_ACCESS-inst.patch [^] (764 bytes) 2013-06-02 22:15 [Show Content]

 Relationships
related to 0014187closedBrad King variable_watch with no command name will cause READ_ACCESS for all variables in the current scope. 

  Notes
(0033172)
Yichao Yu (reporter)
2013-06-02 22:13

looks like the bug is due to a missing entry in the AccessStrings

There are only 6 access names defined in cmVriableWatch.cxx. However, commit 9a5b4eba97628c53f051d653cdc8a301f5f32312 breaks binary compatibility and introduce cmVariableWatch:: UNKNOWN_VARIABLE_DEFINED_ACCESS, which shift all following names by one.
(0033173)
Yichao Yu (reporter)
2013-06-02 22:15

patch attached.
(0033209)
Brad King (manager)
2013-06-03 10:59

Patch applied:

 http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=3d5bb381 [^]

and test added:

 http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=c63d3004 [^]

Thanks!
(0034356)
Robert Maynard (manager)
2013-11-04 09:33

Closing resolved issues that have not been updated in more than 4 months.

 Issue History
Date Modified Username Field Change
2013-06-02 18:24 Yichao Yu New Issue
2013-06-02 22:13 Yichao Yu Note Added: 0033172
2013-06-02 22:15 Yichao Yu File Added: 0001-BUG-0014188-variable_watch-print-REMOVED_ACCESS-inst.patch
2013-06-02 22:15 Yichao Yu Note Added: 0033173
2013-06-03 10:59 Brad King Note Added: 0033209
2013-06-03 10:59 Brad King Assigned To => Brad King
2013-06-03 10:59 Brad King Status new => resolved
2013-06-03 10:59 Brad King Resolution open => fixed
2013-06-03 10:59 Brad King Fixed in Version => CMake 2.8.12
2013-06-03 10:59 Brad King Target Version => CMake 2.8.12
2013-06-03 13:26 Brad King Relationship added related to 0014187
2013-11-04 09:33 Robert Maynard Note Added: 0034356
2013-11-04 09:33 Robert Maynard Status resolved => closed


Copyright © 2000 - 2018 MantisBT Team