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

windres.exe: can't open file `page:': Invalid argument #1850

Closed
3togo opened this issue May 1, 2018 · 5 comments
Closed

windres.exe: can't open file `page:': Invalid argument #1850

3togo opened this issue May 1, 2018 · 5 comments
Labels

Comments

@3togo
Copy link

3togo commented May 1, 2018

I compiled git version of version in msys2 according to the steps posted on the following link
https://wiki.geany.org/howtos/win32/msys2

I ended up have the following errors. Any pointers to fix the problems?

make[3]: Entering directory '/home/Dreli/geanyTemp/geany/src'
windres -i ../geany_private.rc --input-format=rc -o geany_private.res -O coff
../geany_private.rc:2:0: fatal error: when writing output to : Invalid argument

compilation terminated.
C:\msys64\mingw64\bin\windres.exe: can't open file `page:': Invalid argument
C:\msys64\mingw64\bin\windres.exe: preprocessing failed.
make[3]: [Makefile:1505: geany_private.res] Error 1 (ignored)
CCLD geany.exe

@codebrainz
Copy link
Member

Might have better responses by asking question on the mailing list than opening a bug report, more people will see it.

@3togo
Copy link
Author

3togo commented May 6, 2018

the bug fixed by adding --use-temp-file

Dreli:~/geanyTemp/geany/src $ git diff Makefile.am
diff --git a/src/Makefile.am b/src/Makefile.am
index 86c3b7ff..bd8ef8cf 100644
--- a/src/Makefile.am
+++ b/src/Makefile.am
@@ -147,7 +147,7 @@ geany_LDADD += geany_private.res
 WINDRES = $(host_alias)-windres

 geany_private.res: $(top_srcdir)/geany_private.rc
-       $(WINDRES) -i $(top_srcdir)/geany_private.rc --input-format=rc -o $@ -O coff
+       $(WINDRES) -i $(top_srcdir)/geany_private.rc --input-format=rc -o $@ -O coff --use-temp-file

 libgeany_la_SOURCES += win32.c win32.h
 libgeany_la_LIBADD  += -lole32 -lwsock32 -lcomdlg32

winres_use_temp_file.patch.txt

@eht16
Copy link
Member

eht16 commented May 6, 2018

Interesting. On my MSYS2 setup I don't need that option.
But I just tested it and with the temp-file option it compiles also fine, so it is probably a good idea to add it.

Would you mind creating a pull request with the change?

@3togo
Copy link
Author

3togo commented May 6, 2018 via email

@3togo
Copy link
Author

3togo commented May 7, 2018

Some minor issue which is honestly not my concern. Whenever I select "Help"->"About", geany crashed. I am using MSYS2 with x86_64 architecture

@3togo 3togo closed this as completed May 7, 2018
@3togo 3togo reopened this May 7, 2018
@3togo 3togo closed this as completed May 8, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants