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

Missing 'main' option #53

Open
zak-45 opened this issue Jun 14, 2024 · 3 comments
Open

Missing 'main' option #53

zak-45 opened this issue Jun 14, 2024 · 3 comments

Comments

@zak-45
Copy link

zak-45 commented Jun 14, 2024

Hi, I try to build exe file using the multi feature. Doc say need main param, but if I put it I got this warning :

Warning: Unexpected input(s) 'main'.

Here is my action entries:

  - name: Build Executable for ${{runner.os}}
    uses: Nuitka/[email protected]
    with:
      nuitka-version: 2.2.3
      script-name: WLEDVideoSync.py
      company-name: zak-45
      file-version: ${{github.event.inputs.mytag}}
      enable-plugins: tk-inter
      disable-console: True
      deployment: true
      windows-icon-from-ico: assets/favicon.ico
      macos-app-icon: assets/favicon.ico
      macos-create-app-bundle: true
      onefile: true
      onefile-tempdir-spec: ./WLEDVideoSync
      output-file: WLEDVideoSync-${{runner.os}}
      include-module: |
        CastAPI
        zeroconf._utils.ipaddress
        zeroconf._handlers.answers
        pygments.formatters.html
        dbm.dumb
      include-data-dir: |
        tmp=tmp
        log=log
        assets=assets
        config=config
        media=media
        nicegui=nicegui
        xtra=xtra
      include-data-files: |
        favicon.ico=favicon.ico
      main: |
        devstats.py
        netstats.py
        sysstats.py

Thanks

@zak-45 zak-45 changed the title Missing main options Missing 'main' option Jun 14, 2024
@kayhayen
Copy link
Member

It's using script_name so far and requires that even. I agree not having main and not making that one mandatory is kind of a sin, but if you put your list there, I suppose it's going to work. Can you try that?

@zak-45
Copy link
Author

zak-45 commented Jun 14, 2024

That do the trick. thanks

@zak-45 zak-45 closed this as completed Jun 14, 2024
@kayhayen
Copy link
Member

I think it shall become unnecessary though. Cannot allow both due to ordering issues for multidist though.

@kayhayen kayhayen reopened this Jun 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants