nimforum mirror - Nimscript run task after deps installed but before nimble installs

jyapayne (orginal) [2018-06-20T07:23:51+02:00] view original

Hi everyone!

Does anyone know how to make nimscript run a task that's basically after the dependencies are installed, but before the current library is installed by nimble? My use case is the following:

# Package

version       = "0.1.0"
author        = "Joey Payne"
description   = "Nintendo Switch library libnx for Nim."
license       = "The Unlicense"

srcDir = "src"

# Deps
requires "nim >= 0.18.0", "https://github.com/jyapayne/nimgenEx#head"

task setup, "Download and generate":
  exec "nimgen libnxGen.cfg" # I want this to run after deps are installed because nimgen doesn't exist yet

before install:
  setupTask()

When I run it now, I get an error:


sh: 1: nimgen: not found
stack trace: (most recent call last)
home/joey/Projects/nim-libnx/libnx.nimble(17) installBefore
home/joey/Projects/nim-libnx/libnx.nimble(14) setupTask
home/joey/Nim/lib/system/nimscript.nim(237) exec
home/joey/Nim/lib/system/nimscript.nim(237, 7) Error: unhandled exception: FAILED: nimgen libnxGen.cfg

because obviously nimgen doesn't exist yet because it's not installed. However, if I first install it, it works fine. But I don't think that's a reasonable flow to first ask the user to install nimgen and then use the library.

Is there an option in nimscript I'm missing?

DeletedUser (orginal) [2018-06-20T08:57:59+02:00] view original

You could import and use nimgen in the script but it would make nimgen run in the VM. Also it seems nimgen doesn't have a when isMainModule check so importing it wouldn't exactly work.

dom96 (orginal) [2018-06-20T11:49:34+02:00] view original

There is already an issue when using nimgen with Nimble due to a bug in Nimble with how it handles the before install tasks. I guess this is another bug that we need to fix.

Here is the relevant PR that fixes another bug related to this: https://github.com/nim-lang/nimble/pull/464

jyapayne (orginal) [2018-06-20T11:56:57+02:00] view original

It looks like that's the same issue, actually. Is there any reason why the fix wasn't pulled in? Just failing build?

jyapayne (orginal) [2018-06-20T11:59:45+02:00] view original

I'd like it not to run in the VM, since it uses various things not available in the VM. I've made my own fork of nimgen at https://github.com/jyapayne/nimgenEx that adds a bunch of features that I needed when porting libnx. I tried to make pull requests to the original repo, but @genotrance hasn't responded to them as of yet.

dom96 (orginal) [2018-06-20T12:35:31+02:00] view original

Mainly I just didn't have the time to review it properly yet. I usually commit a certain amount of time to Nimble and then take care of all outstanding PRs at once instead of doing it progressively. A new Nim release will likely happen soon so I will take care of it then.

jyapayne (orginal) [2018-06-20T12:59:42+02:00] view original

Alright, sounds good :) Especially about the new Nim release!

Mirror of forum.nim-lang.org

3949 :: Nimscript run task after deps installed but before nimble installs