{"push_id":597473564,"size":11,"distinct_size":11,"ref":"refs/heads/master","head":"22ae7ce8292f18466dc0c18a2a7c763d9972b983","before":"53606463c10c4780ac7c05d1d7167f7c3147e08e","commits":[{"sha":"f25a558394a70980a83db9dafa7ea5746441145c","author":{"email":"f955cf5f42dff0db8f444c4740a67a0cf0b4805d@users.noreply.github.com","name":"Lydia Duncan"},"message":"Set up structure for isMount\n\nAdds it to the FileSystem module and a bit of the runtime set up (though as the\ncode stands today, it will not build because that function doesn't return\nanything).","distinct":true,"url":"https://api.github.com/repos/chapel-lang/chapel/commits/f25a558394a70980a83db9dafa7ea5746441145c"},{"sha":"b915351a6dd4ff1e8ba5e4f47e2b913fdac292d6","author":{"email":"f955cf5f42dff0db8f444c4740a67a0cf0b4805d@users.noreply.github.com","name":"Lydia Duncan"},"message":"Attempt to get it to work.\n\nStill need to figure out if this fits within our runtime story as it stands.","distinct":true,"url":"https://api.github.com/repos/chapel-lang/chapel/commits/b915351a6dd4ff1e8ba5e4f47e2b913fdac292d6"},{"sha":"6d580df4a15a67f9b5f4a8ab968d5d60f21f3f88","author":{"email":"f955cf5f42dff0db8f444c4740a67a0cf0b4805d@users.noreply.github.com","name":"Lydia Duncan"},"message":"Change to runtime implementation\n\ngbtitus suggested adding a new memory descriptor for my case, so we can tell\nthem apart.  mount uses CHPL_RT_MD_OS_LAYER_TMP_DATA now.","distinct":true,"url":"https://api.github.com/repos/chapel-lang/chapel/commits/6d580df4a15a67f9b5f4a8ab968d5d60f21f3f88"},{"sha":"b646a1f558861de72cb1cdaccb8f6368315ac81e","author":{"email":"f955cf5f42dff0db8f444c4740a67a0cf0b4805d@users.noreply.github.com","name":"Lydia Duncan"},"message":"Make the mount point a more common one, /dev\n\nThanks, Kyle!","distinct":true,"url":"https://api.github.com/repos/chapel-lang/chapel/commits/b646a1f558861de72cb1cdaccb8f6368315ac81e"},{"sha":"6b06692d8469566765f8c03821e69872a9660e4d","author":{"email":"f955cf5f42dff0db8f444c4740a67a0cf0b4805d@users.noreply.github.com","name":"Lydia Duncan"},"message":"Merge branch 'master' into isMount","distinct":true,"url":"https://api.github.com/repos/chapel-lang/chapel/commits/6b06692d8469566765f8c03821e69872a9660e4d"},{"sha":"715979797af2aa56f1f1c61ba703d11edfe0fabc","author":{"email":"f955cf5f42dff0db8f444c4740a67a0cf0b4805d@users.noreply.github.com","name":"Lydia Duncan"},"message":"Update to handle root case.\n\nA root directory will have the same st_dev, but roots are by definition mount\npoints.  However, in this case, its parent will be itself.  I think.  Need to\ntest on cygwin","distinct":true,"url":"https://api.github.com/repos/chapel-lang/chapel/commits/715979797af2aa56f1f1c61ba703d11edfe0fabc"},{"sha":"b3345b255663acffa72497880986967930c7486b","author":{"email":"f955cf5f42dff0db8f444c4740a67a0cf0b4805d@users.noreply.github.com","name":"Lydia Duncan"},"message":"Update comment.","distinct":true,"url":"https://api.github.com/repos/chapel-lang/chapel/commits/b3345b255663acffa72497880986967930c7486b"},{"sha":"dcd6412ff74a002bdb0a5326eb0d09e203371865","author":{"email":"f955cf5f42dff0db8f444c4740a67a0cf0b4805d@users.noreply.github.com","name":"Lydia Duncan"},"message":"Add a new test for the root case.\n\nThe /dev case doesn't apply to cygwin, but it seems useful to check both the\nroot and another known mount point (because they are handled distinctly), so\ninstead of making work for myself later, I just copied the test over and\nreplaced \"/dev\" with just \"/\".  Now we can check that cygwin detects something\nright, and I'll expand testing further in the future.","distinct":true,"url":"https://api.github.com/repos/chapel-lang/chapel/commits/dcd6412ff74a002bdb0a5326eb0d09e203371865"},{"sha":"2d0c222a2f3d3f03e5651306a1bc7a76e1ab0ebd","author":{"email":"f955cf5f42dff0db8f444c4740a67a0cf0b4805d@users.noreply.github.com","name":"Lydia Duncan"},"message":"Exit with false if the argument name is a file.\n\nMount points are directories.","distinct":true,"url":"https://api.github.com/repos/chapel-lang/chapel/commits/2d0c222a2f3d3f03e5651306a1bc7a76e1ab0ebd"},{"sha":"188c74615d743f734356717c818a6b48fa79e283","author":{"email":"f955cf5f42dff0db8f444c4740a67a0cf0b4805d@users.noreply.github.com","name":"Lydia Duncan"},"message":"Take Greg's comments into account.\n\nStrengthened the comment and used allocMany instead of allocManyZero.","distinct":true,"url":"https://api.github.com/repos/chapel-lang/chapel/commits/188c74615d743f734356717c818a6b48fa79e283"},{"sha":"22ae7ce8292f18466dc0c18a2a7c763d9972b983","author":{"email":"f955cf5f42dff0db8f444c4740a67a0cf0b4805d@users.noreply.github.com","name":"Lydia Duncan"},"message":"Merge pull request #1531 from lydia-duncan/isMount\n\nImplement isMount [reviewed by gbtitus]\r\n\r\n\r\n\r\nIt returns true if the name of a mount point was sent to it, false\r\notherwise. Like the other is* functions, if the name provided does\r\nnot refer to something that exists, error out.\r\n\r\nDesign decisions: Added a new memory descriptor for clearer error\r\nhandling (CHPL_RT_MD_OS_LAYER_TMP_DATA). Compared the\r\nstat.st_dev number of the directory and its parent - if they weren't the\r\nsame, we had a mount point. This would ignore the root case, so\r\nalso checked to make sure they didn't refer to the same directory. I\r\nalso exited before entering runtime code if the name referred to a file\r\ninstead of a directory, since files can't be mount points.\r\n\r\nThere are two tests: one whether \"/\" is a mount point and one if \"/dev\"\r\nis valid. I added both because \"/dev\" wasn't present on cygwin, and\r\n\"/\" is treated differently in my implementation because it is a root\r\ndirectory.  I will add more tests during code freeze","distinct":true,"url":"https://api.github.com/repos/chapel-lang/chapel/commits/22ae7ce8292f18466dc0c18a2a7c763d9972b983"}]}