error: extra argument 'requestTimeout' in call requestTimeout: self.connectionPoolTimeout,

oprisk commented 4 years ago

Invalid Call: /Users/paolo/mtgDEV/_Server/test/TestServer/myProject/.build/checkouts/fluent-mysql-driver/Sources/FluentMySQLDriver/FluentMySQLConfiguration.swift:130:34: error: extra argument 'requestTimeout' in call requestTimeout: self.connectionPoolTimeout,


[1588/1619] Compiling FluentMySQLDriver FluentMySQLDatabase.swift

-----------------
Package.swift:
// swift-tools-version:5.2
import PackageDescription

let package = Package(
    name: "myProject",
    platforms: [
       .macOS(.v10_15)
    ],
    dependencies: [
        // 💧 A server-side Swift web framework.
        .package(url: "https://github.com/vapor/vapor.git", from: "4.14.0"),
        .package(url: "https://github.com/vapor/leaf.git", from: "4.0.0-rc"),
        .package(url: "https://github.com/vapor/fluent.git", from: "4.0.0"),
        .package(url: "https://github.com/vapor/fluent-mysql-driver.git", from: "4.0.0")
    ],
    targets: [
        .target(name: "App", dependencies: [
            .product(name: "Leaf", package: "leaf"),
            .product(name: "Fluent", package: "fluent"),
            .product(name: "FluentMySQLDriver", package: "fluent-mysql-driver"),
            .product(name: "Vapor", package: "vapor"),
        ]),
        .target(name: "Run", dependencies: ["App"]),
        .testTarget(name: "AppTests", dependencies: [
            .target(name: "App"),
            .product(name: "XCTVapor", package: "vapor"),
        ])
    ]
)

0xTim commented 4 years ago

If you update to the latest package versions does this error go away?

oprisk commented 4 years ago

Sorry for the delay. I updated to version 4.0.0-rc and error went away: from: .package(url: "https://github.com/vapor/fluent-mysql-driver.git", from: "4.0.0") to: .package(url: "https://github.com/vapor/fluent-sqlite-driver.git", from: "4.0.0-rc")

Presumably, "-rc" is the latest beta?

0xTim commented 4 years ago

Are you switching to SQLite? There aren't any betas any more for the Fluent stuff so just 4.0.0 should work after a package update (swift package update from the command line)

oprisk commented 4 years ago

The issue is with MySQL driver, (disregard SQLite, it works just fine using "-rc" release, but not 4.0.0.)

I tested both MySQLdriver 4.0.0 and 4.0.0-rc, with the same error: /Users/paolo/_MtgDEV/_Server/MtgServer/.build/checkouts/fluent-mysql-driver/Sources/FluentMySQLDriver/FluentMySQLConfiguration.swift:130:34: error: extra argument 'requestTimeout' in call requestTimeout: self.connectionPoolTimeout,

I'm able to comment the extra parameter in the call, provided the project is opened via Xcode project, unable to change package opening the project in Xcode opening the package.swift (as they are dynamically linked packages)

0xTim commented 4 years ago

Can you provide your package.resolved?

oprisk commented 4 years ago


        let
/Users/paolo/_MtgDEV/_Server/MtgServer/.build/checkouts/swift-crypto/Sources/Crypto/Util/BoringSSL/ArbitraryPrecisionInteger_boring.swift:196:13: warning: variable 'bnCtx' was never mutated; consider changing to 'let' constant
        var bnCtx = CCryptoBoringSSL_BN_CTX_new()!
        ~~~ ^
        let
/Users/paolo/_MtgDEV/_Server/MtgServer/.build/checkouts/swift-crypto/Sources/Crypto/Util/BoringSSL/ArbitraryPrecisionInteger_boring.swift:196:13: warning: variable 'bnCtx' was never mutated; consider changing to 'let' constant
        var bnCtx = CCryptoBoringSSL_BN_CTX_new()!
        ~~~ ^
        let
/Users/paolo/_MtgDEV/_Server/MtgServer/.build/checkouts/swift-crypto/Sources/Crypto/Util/BoringSSL/ArbitraryPrecisionInteger_boring.swift:196:13: warning: variable 'bnCtx' was never mutated; consider changing to 'let' constant
        var bnCtx = CCryptoBoringSSL_BN_CTX_new()!
        ~~~ ^
        let
/Users/paolo/_MtgDEV/_Server/MtgServer/.build/checkouts/swift-crypto/Sources/Crypto/Util/BoringSSL/ArbitraryPrecisionInteger_boring.swift:196:13: warning: variable 'bnCtx' was never mutated; consider changing to 'let' constant
        var bnCtx = CCryptoBoringSSL_BN_CTX_new()!
        ~~~ ^
        let
/Users/paolo/_MtgDEV/_Server/MtgServer/.build/checkouts/swift-crypto/Sources/Crypto/Util/BoringSSL/ArbitraryPrecisionInteger_boring.swift:196:13: warning: variable 'bnCtx' was never mutated; consider changing to 'let' constant
        var bnCtx = CCryptoBoringSSL_BN_CTX_new()!
        ~~~ ^
        let
/Users/paolo/_MtgDEV/_Server/MtgServer/.build/checkouts/fluent-mysql-driver/Sources/FluentMySQLDriver/FluentMySQLConfiguration.swift:130:34: error: extra argument 'requestTimeout' in call
            requestTimeout: self.connectionPoolTimeout,
                            ~~~~~^~~~~~~~~~~~~~~~~~~~~
/Users/paolo/_MtgDEV/_Server/MtgServer/.build/checkouts/fluent-mysql-driver/Sources/FluentMySQLDriver/FluentMySQLConfiguration.swift:130:34: error: extra argument 'requestTimeout' in call
            requestTimeout: self.connectionPoolTimeout,
                            ~~~~~^~~~~~~~~~~~~~~~~~~~~
[1524/1617] Compiling FluentMySQLDriver FluentMySQLDatabase.swift

oprisk commented 4 years ago

disregard, out of sync with Xcode compile, re-running again

0xTim commented 4 years ago

Run swift package update - you're pulling in an old version of Fluent MySQL Driver, you want 4.0.0 https://github.com/vapor/fluent-mysql-driver/releases/tag/4.0.0

oprisk commented 4 years ago

I ran swift package reset, swift package resolve, swift build I will redo with swift package update

oprisk commented 4 years ago

without package update:


        let
/Users/paolo/_MtgDEV/_Server/MtgServer/.build/checkouts/swift-crypto/Sources/Crypto/Util/BoringSSL/ArbitraryPrecisionInteger_boring.swift:196:13: warning: variable 'bnCtx' was never mutated; consider changing to 'let' constant
        var bnCtx = CCryptoBoringSSL_BN_CTX_new()!
        ~~~ ^
        let
/Users/paolo/_MtgDEV/_Server/MtgServer/.build/checkouts/swift-crypto/Sources/Crypto/Util/BoringSSL/ArbitraryPrecisionInteger_boring.swift:196:13: warning: variable 'bnCtx' was never mutated; consider changing to 'let' constant
        var bnCtx = CCryptoBoringSSL_BN_CTX_new()!
        ~~~ ^
        let
/Users/paolo/_MtgDEV/_Server/MtgServer/.build/checkouts/swift-crypto/Sources/Crypto/Util/BoringSSL/ArbitraryPrecisionInteger_boring.swift:196:13: warning: variable 'bnCtx' was never mutated; consider changing to 'let' constant
        var bnCtx = CCryptoBoringSSL_BN_CTX_new()!
        ~~~ ^
        let
/Users/paolo/_MtgDEV/_Server/MtgServer/.build/checkouts/swift-crypto/Sources/Crypto/Util/BoringSSL/ArbitraryPrecisionInteger_boring.swift:196:13: warning: variable 'bnCtx' was never mutated; consider changing to 'let' constant
        var bnCtx = CCryptoBoringSSL_BN_CTX_new()!
        ~~~ ^
        let
/Users/paolo/_MtgDEV/_Server/MtgServer/.build/checkouts/swift-crypto/Sources/Crypto/Util/BoringSSL/ArbitraryPrecisionInteger_boring.swift:196:13: warning: variable 'bnCtx' was never mutated; consider changing to 'let' constant
        var bnCtx = CCryptoBoringSSL_BN_CTX_new()!
        ~~~ ^
        let
/Users/paolo/_MtgDEV/_Server/MtgServer/.build/checkouts/fluent-mysql-driver/Sources/FluentMySQLDriver/FluentMySQLConfiguration.swift:130:34: error: extra argument 'requestTimeout' in call
            requestTimeout: self.connectionPoolTimeout,
                            ~~~~~^~~~~~~~~~~~~~~~~~~~~
/Users/paolo/_MtgDEV/_Server/MtgServer/.build/checkouts/fluent-mysql-driver/Sources/FluentMySQLDriver/FluentMySQLConfiguration.swift:130:34: error: extra argument 'requestTimeout' in call
            requestTimeout: self.connectionPoolTimeout,
                            ~~~~~^~~~~~~~~~~~~~~~~~~~~
[1549/1619] Compiling Vapor JSONCoder+Custom.swift
McBkPaolo.local /Users/paolo/_MtgDEV/_Server/MtgServer %

oprisk commented 4 years ago

The swift package update compiles correctly, without error Not sure what originally caused the issue, instance of package manager bug?

oprisk commented 4 years ago

this is the package.swift:

// swift-tools-version:5.2 import PackageDescription

let package = Package( name: "MtgServer", platforms: [ .macOS(.v10_15) ], dependencies: [ // 💧 A server-side Swift web framework. .package(url: "https://github.com/vapor/vapor.git", from: "4.14.0"), .package(url: "https://github.com/vapor/leaf.git", from: "4.0.0-rc"), .package(url: "https://github.com/vapor/fluent.git", from: "4.0.0"), .package(url: "https://github.com/vapor/fluent-mysql-driver.git", from: "4.0.0") ], targets: [ .target(name: "App", dependencies: [ .product(name: "Leaf", package: "leaf"), .product(name: "Fluent", package: "fluent"), .product(name: "FluentMySQLDriver", package: "fluent-mysql-driver"), .product(name: "Vapor", package: "vapor"), ]), .target(name: "Run", dependencies: ["App"]), .testTarget(name: "AppTests", dependencies: [ .target(name: "App"), .product(name: "XCTVapor", package: "vapor"), ]) ] )

oprisk commented 4 years ago

Thank-you for the assistance

0xTim commented 4 years ago

The bug was caused by an update somewhere that FluentMySQL didn't specify when it should have done. Until you run a package update it will stick with the versions in the resolved file (which in your case were out of date). Glad it's working!

grundoon commented 4 years ago

I'm a little troubled by the fact that your swift-nio (thus swift-nio-ssl and swift-nio-http2 as well) are still months out of date.

oprisk commented 4 years ago

how can I force to pull the latest, if swift package update is not doing this?

grundoon commented 4 years ago

It really should be (and that's a problem) I think... but I'd try blowing away the build and DerivedData directories, and manually changing NIO to from: "2.23.0" in vapor/Package.swift.

oprisk commented 4 years ago

using MacBook, running on Big Sur latest beta deleted project build folder cleaned ~/Library/Developer/Xcode/DerivedData ran package reset, update

-- it did pull NIO 2.23.0, see below:

vapor / fluent-mysql-driver

error: extra argument 'requestTimeout' in call requestTimeout: self.connectionPoolTimeout, #197