search

lighterowl / transgui

A fork of Transmission Remote GUI
GNU General Public License v2.0
152 stars 4 forks source link

httpsend #6

Closed hallonstedt closed 1 year ago

hallonstedt commented 1 year ago

I may be doing this wrong but I am trying to compile the binary on Windows 10 x64. I unpack the files, successfully run install_deps.bat and then when I run make_setup.bat I get the following error:

(3104) Compiling main.pas
E:\transgui-master\main.pas(46,65) Warning: (3226) Don't load LINEINFO unit manually, Use the -gl compiler switch instead
E:\transgui-master\main.pas(44,3) Fatal: (10022) Can't find unit httpsend used by Main
Fatal: (1018) Compilation aborted

Any suggestions on how to resolve this?

lighterowl commented 1 year ago

synapse is now a submodule. Make sure you do git submodule update --init --recursive after cloning the main repository.

Also, those scripts will not work as they still expect old makefiles to be in place. Instead, have a look at .github/build_windows.ps1 or just download the latest action artifact. :-)

hallonstedt commented 1 year ago

Thank you!

I have no clue how to compile on windows so now I'm happy. It still says version 5.18.0 but the compiler versions are updated so I think we're good. You may want to add a version tag to indicate it's your artifact.

You may also want to mention there are pre-compiled versions under Actions / windows build in your README.md. To a novice, it's not obvious as we usually only look for published releases.

lighterowl commented 1 year ago

I will post them as releases once they're ready. This should still be considered pre-release even though, as you've noticed, the compilation is already done with newer FPC and Lazarus versions.