Issue 11799 - Incompatible argument types in create_dmd_release
Summary: Incompatible argument types in create_dmd_release
Status: RESOLVED FIXED
Alias: None
Product: D
Classification: Unclassified
Component: installer (show other issues)
Version: D2
Hardware: x86_64 All
: P2 normal
Assignee: No Owner
URL:
Keywords: pull
Depends on:
Blocks:
 
Reported: 2013-12-22 04:51 UTC by Andrew Edwards
Modified: 2014-02-04 14:17 UTC (History)
2 users (show)

See Also:

Attachments
modified to address type incompatability on line 1503 (47.04 KB, application/octet-stream)
2013-12-22 04:51 UTC, Andrew Edwards 		Details
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description Andrew Edwards 2013-12-22 04:51:47 UTC 
Created attachment 1303 [details]
modified to address type incompatability on line 1503

create_dmd_release.d(1503): Error: function core.sys.posix.sys.stat.chmod
(const(char*), ushort) is not callable using argument types (immutable(char)*,
uint)

Line 1503: setFileAttributes(toStringz(path), am.fileAttributes);

The following works:
     setFileAttributes(toStringz(path), to!ushort(am.fileAttributes));

Whether that's the correct fix or not I'm not sure.
Comment 1 Jacob Carlborg 2013-12-27 07:41:51 UTC 
If you change the signature to "const(char)*", does that work? This is not a regression create_dmd_release is a new tool.
Comment 3 github-bugzilla 2014-01-13 14:53:10 UTC 
Commits pushed to master at https://github.com/D-Programming-Language/installer

https://github.com/D-Programming-Language/installer/commit/113df7987e2d1b1cb252cd38be677a9b8148c63f
fix Issue 11799 - Incompatible argument types in create_dmd_release

- use std.file.setAttributes

https://github.com/D-Programming-Language/installer/commit/848b0b65e20e1c5e32fb8ba6caeac332084fb981
Merge pull request #33 from MartinNowak/fix11799

[2.065] fix Issue 11799 - Incompatible argument types in create_dmd_release