vilmibm / gh-contribute

Small gh extension that suggests issues to work on in a given GitHub repository
GNU General Public License v3.0
29 stars 6 forks source link

Failing on NixOS #7

Closed maxdevjs closed 2 years ago

maxdevjs commented 2 years ago

I found three issues on my system (NixOS) related to this extension so far:

This is somewhat weird as, for example, gh-screensaver works after replacing the shebang.

At the moment, I do not know what the difference between the two extensions is. Especially, I am not sure why gh-screensaver works even if the two install scripts are basically the same!

Currently gh-contribute:

I know zero of Go and Go on NixOS then I can not assure that my assumption is right (principally why this extension executable should, apparently, be patched).

But, I changed the logic of gh-contribute and I can now successfully build from an initial gh contribute and the executable runs. This should prove that the downloaded extension does not work because not patched(!), not built for NixOS.

I will clean the script and post for review.

I also checked

Any suggestion about what part could influence the behavior, is welcome.

Anyway:

maxdevjs commented 2 years ago

I confirm that some (and not only) Go extensions can fail on NixOS (and most likely, on other non FHS compliant systems - perhaps Guix).

Following a NixOS case (all the extensions should be compiled locally to be 'patched' and consequenly work). Right now the installation script:

Okay, at the moment I have no idea how this extension world works: if there is or not a source of truth to which every extension author would refer.

Thereby I chose @vilmibm as an author for this brainstorm.

The installation script(s) should be kindly patched. And the word spread :)

If there will be interest in solving/soften the situation, I will open an issue proposing a slightly different script workflow.

vilmibm commented 2 years ago

Thanks for the thorough debugging. I've moved this to use the newer precompilation strategy so there is no more glue bash script to contend with. In theory this should help, so I'm closing this issue.