Bugs item #3032628, was opened at 2010-07-21 16:04
Message generated for change (Comment added) made by dogu
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=402868&aid=3032628&group_id=31650

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Tasks
Group: 0.91-alpha1
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Bogdan P (lica-mobilu)
Assigned to: Nobody/Anonymous (nobody)
Summary: <sysinfo /> on x64 systems

Initial Comment:
On my W7x64 task <sysinfo /> does not work. Looking with reflector the error 
comes from:


public class PropertyDictionary : DictionaryBase
  - private static void ValidatePropertyName(string propertyName, Location 
location)

Output by Nant is:

Property name 'sys.env.CommonProgramFiles(x86)' is invalid.:
NAnt.Core.BuildException: Property name 'sys.env.CommonProgramFiles(x86)' is inv
alid.
   at NAnt.Core.PropertyDictionary.ValidatePropertyName(String propertyName, Loc
ation location)
   at NAnt.Core.PropertyDictionary.OnValidate(Object key, Object value)
   at System.Collections.DictionaryBase.System.Collections.IDictionary.set_Item(
Object key, Object value)
   at NAnt.Core.PropertyDictionary.set_Item(String name, String value)
   at NAnt.Core.Tasks.SysInfoTask.ExecuteTask()
   at NAnt.Core.Task.Execute()
   at NAnt.Core.Project.InitializeProjectDocument(XmlDocument doc)
   at NAnt.Core.Project.Execute()
   at NAnt.Core.Project.Run()

This show that this line of code from method above raises an error:


    if (!Regex.IsMatch(propertyName, @"^[_A-Za-z0-9][_A-Za-z0-9\-.]*$"))
    {
        throw new BuildException(string.Format(CultureInfo.InvariantCulture, 
ResourceUtils.GetString("NA1064"), new object[] { propertyName }), location);
    }

Solution:

Allow ( and ) in property name.

----------------------------------------------------------------------

Comment By: Dominik Guder (dogu)
Date: 2010-07-31 06:31

Message:
Hi,
I stubled about this issue some other day before releasing 0.90. 
The reason is the function parser which needs the parens as separator to
work correctly. Therefore they are excluded from property names.
I did some checkings like renaming ( and ) to _. But finally we did not
came to an acceptable  solution what to do now and posponed the issue.

Until we grab this issue again, the only way is to set failonerror="false"
on sysinfo task.

So far Dominik

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=402868&aid=3032628&group_id=31650

------------------------------------------------------------------------------
The Palm PDK Hot Apps Program offers developers who use the
Plug-In Development Kit to bring their C/C++ apps to Palm for a share
of $1 Million in cash or HP Products. Visit us here for more details:
http://p.sf.net/sfu/dev2dev-palm
_______________________________________________
nant-developers mailing list
nant-developers@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nant-developers

Reply via email to