Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sage does not build on vfat filesystems #24953

Closed
jdemeyer opened this issue Mar 12, 2018 · 6 comments
Closed

Sage does not build on vfat filesystems #24953

jdemeyer opened this issue Mar 12, 2018 · 6 comments

Comments

@jdemeyer
Copy link

This check in build/bin/sage-spkg should really depend on the filesystem and not on the OS:

                msg="$script should not be marked executable in the build/pkgs directory"
                if [ "$UNAME" = "CYGWIN" ]; then
                    # On Cygwin we can't necessarily rely on file permissions
                    # being sane, so just issue a warning; on other platforms
                    # this should be enforced as an error
                    echo >&2 "WARNING: $msg"
                else
                    error_msg "$msg"
                    exit 1
                fi

CC: @embray

Component: build

Issue created by migration from https://trac.sagemath.org/ticket/24953

@jdemeyer jdemeyer added this to the sage-8.2 milestone Mar 12, 2018
@saraedum
Copy link
Member

comment:1

I took the liberty to downgrade this. I don't see why this would be critical. Feel free to set it back.

Of course this check is wrong in theory, but at the moment I feel it's good enough. We could change the comment by replacing "Cygwin" with "vfat & friends" but it's not worth to figure out the file system from my point of view.

@embray
Copy link
Contributor

embray commented Mar 30, 2018

comment:2

I think also lots of other things will go wrong if you try to build on a VFAT filesystem.

@saraedum
Copy link
Member

saraedum commented Apr 1, 2018

comment:3

If jdemeyer agrees, we should close this.

@saraedum saraedum removed this from the sage-8.2 milestone Apr 1, 2018
@jdemeyer
Copy link
Author

jdemeyer commented Apr 1, 2018

comment:4

Replying to @embray:

I think also lots of other things will go wrong if you try to build on a VFAT filesystem.

Like symlinks I guess?

@jdemeyer
Copy link
Author

comment:5

Duplicate of #25207

@embray
Copy link
Contributor

embray commented Apr 19, 2018

comment:6

I guess really #25207 could be considered a duplicate of this ticket, albeit a bit broader in scope.

I'm still not sure about FAT if you can't have symlinks though...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants