Message ID | DM6PR19MB2460B6C6C5A1E4B1E98CDEAE997B9@DM6PR19MB2460.namprd19.prod.outlook.com |
---|---|
State | New |
Headers | show |
Series | [bug#52238] gnu: Add MEGA SDK | expand |
Context | Check | Description |
---|---|---|
cbaines/comparison | success | View comparision |
cbaines/git branch | success | View Git branch |
cbaines/applying patch | success | View Laminar job |
cbaines/issue | success | View issue |
Hi, Am Montag, dem 20.12.2021 um 01:58 +0000 schrieb Jaft: > Makes sense; it seems like it's present in the integration and > tool_purge_account tests while the third group of tests – unit – > seems to not rely on those. I adjusted the files to remove those two > sets of tests and things built alright. LGTM, albeit somewhat overkill. Wouldn't it suffice to simply set TESTS and possibly dropping those extra subdirectories? > Gotcha. This is really useful and helpful; thanks a ton for walking > through it. > > I've attached two patches, one for each package; the SDK one removes > the troublesome tests so the rest can be ran. By patch, I do mean patches in mbox format, not mere `git diff`s. Don't strip mail headers; those are important. Cheers
* gnu/packages/sync.scm (megacmd): Use MEGA SDK --- gnu/packages/sync.scm | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/gnu/packages/sync.scm b/gnu/packages/sync.scm index ce815ed5c7..1c75c3ca53 100644 --- a/gnu/packages/sync.scm +++ b/gnu/packages/sync.scm @@ -222,8 +222,7 @@ (define-public megacmd (method git-fetch) (uri (git-reference (url "https://github.com/meganz/MEGAcmd") - (commit (string-append version "_Linux")) - (recursive? #t))) + (commit (string-append version "_Linux")))) (sha256 (base32 "004j8m3xs6slx03g2g6wzr97myl2v3zc09wxnfar5c62a625pd53")) @@ -242,6 +241,7 @@ (define-public megacmd ("curl" ,curl) ("freeimage" ,freeimage) ("gtest" ,googletest) + ("mega-sdk" ,mega-sdk) ("openssl" ,openssl) ("pcre" ,pcre) ("readline" ,readline)