Bug creation and email sending has been disabled, file new bugs at gcc.gnu.org/bugzilla

Bug 5

Summary:	std.file: struct_stat64 wrong for ARM
Product:	GDC	Reporter:	Johannes Pfau <johannespfau>
Component:	libgphobos	Assignee:	Johannes Pfau <johannespfau>
Status:	RESOLVED FIXED
Severity:	trivial	CC:	ibuclaw, johannespfau
Priority:	---
Version:	development
Hardware:	ARM
OS:	Linux

Description Johannes Pfau 2012-09-24 14:58:29 CEST

The struct_stat64 declaration forces align(4) on st_size. This is wrong for ARM, I wonder if it's actually correct for x86? Should be trivial to fix, but std.file probably should define this struct at all. It should use core.sys.posix.sys.stat instead, which currently does not have a struct_stat64 definition.

Filed here, so it won't be forgotten.

Comment 1 Iain Buclaw 2012-09-24 15:51:38 CEST

This should first be fixed in phobos - which should not be defining it's own version of struct statfs at all.  It should instead use the druntime version.

There is an open ticket somewhere in D.bugs - flag it again and raise a pull request to remove it entirely, and update any parts that may break as a result.

Comment 2 Johannes Pfau 2012-09-28 18:04:47 CEST

https://github.com/D-Programming-Language/phobos/pull/819

Got rid off all the old *64 functions. (struct_stat64 is actually deprecated). I still have to check whether druntime's stat_t is correct though.

Comment 3 Iain Buclaw 2013-03-28 11:32:28 CET

I do believe this is fixed now.