oomek / attractplus

Attract-Mode Plus
GNU General Public License v3.0
41 stars 14 forks source link

MacOSx64 compiles, but create-pkg.sh fails #28

Closed estefan3112 closed 1 year ago

estefan3112 commented 1 year ago

UPDATE: It seems to me that you only need to accept the create-pkg.sh file from DJHAN's fork, because that one works. For testing purposes, I copied DJHAN's entire osx folder to your repository, and the script worked.

Hello, while the source builds nicely on an Intel Mac, the creation of the app bundle fails with the following error message:

cp: ../attract: No such file or directory ERROR in ./create-pkg.sh : line 73 exit code 1

This is apparently because the cloned directory is called attract plus instead of attract. After correcting this, the script fails with the following error:

fatal: Not a valid object name v2.6.2^ fatal: ambiguous argument 'v2.6.2..HEAD': unknown revision or path not in the working tree. Use '--' to separate paths from revisions, like this: 'git [...] -- [...]' fatal: ambiguous argument '..v2.6.2': unknown revision or path not in the working tree. Use '--' to separate paths from revisions, like this: 'git [...] -- [...]' env: python: No such file or directory ERROR in ./create-pkg.sh : line 103 exit code 127

That one I could not figure out by myself.

Cheers!

zpaolo11x commented 1 year ago

The script create-pkg.sh is obsolete and doesn't work for the current builds AFAIK. The script that CI is using to build packages is appbuilder.sh.

On a side note, why you need to compile yourself, instead of using the pre-built artifacts?

estefan3112 commented 1 year ago

Hello and thanks! I started compiling on my M1 Mac, as the x64 builds from your repository do not work. On Apple Silicon, your repository still does not compile, only DJHAN's for does (EDIT: He did a rather rough fix of the ljpeg dependency, with that one, your repository also compiles under Apple Silicon). The osx folder, as amended by DJHAN, creates nice DMGs for both platforms, also x64 in your repository, that's I wanted to tell you. In case of Apple Silicon, own codesigining is required to have them run. Cheers!

estefan3112 commented 1 year ago

Let's close this finally 😀