Skip to content

persistent-sqlite-2.5.0.1 fails on Mac OS X #564

Closed
Closed
persistent-sqlite-2.5.0.1 fails on Mac OS X#564

Description

@borsboom
borsboom
opened on May 30, 2016

Opening a database fails with:

sqlite-test: SQLite3 returned ErrorCan'tOpen while attempting to perform open "_test.sqlite3".

Minimal reproducing program:

{-# LANGUAGE OverloadedStrings #-}
module Main where

import qualified Database.Sqlite as Sqlite

main :: IO ()
main = do
  _ <- Sqlite.open "_test.sqlite3"
  return ()

It looks like the persistent-sqlite tests don't trigger this issue because they use the :memory: connection which doesn't use the stat system call.

(previously mentioned in #561 (comment))

Activity

  1. borsboom commented on May 30, 2016

    @borsboom
    borsboom
    on May 30, 2016
    ContributorAuthor

    Looks like the problem is that this application of the patch (0df11d7) adds an lstat_local that was not needed before (in previous versions of the patch like ac45d80), but the type signature is wrong. It should match stat's type signature (taking a const char * as the first argument) but instead matches fstat's (taking an int).

  2. borsboom
    added 3 commits that reference this issue on May 30, 2016

    Fix workaround for stat64 bug on OS X (yesodweb#564)

    45da8f8

    Fix workaround for stat64 bug on OS X (yesodweb#564)

    36f38db

    Fix workaround for stat64 bug on OS X (fixes yesodweb#564)

    d5836ed
  3. snoyberg
    closed this as completedin d5836edon May 31, 2016
  4. snoyberg
    added a commit that references this issue on May 31, 2016

    Merge pull request #565 from borsboom/pr/564

    6e46c2c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions