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

Multiple warnings with valac 0.39.7 #20

Closed
martinkg opened this issue Mar 14, 2018 · 10 comments
Closed

Multiple warnings with valac 0.39.7 #20

martinkg opened this issue Mar 14, 2018 · 10 comments
Assignees
Labels
type: bug Something is not working as expected or the documentation is incorrect.

Comments

@martinkg
Copy link

martinkg commented Mar 14, 2018

Hi,

when compiling diorite-4.10.0 on Fedora 28, i get this errors:

Compilation succeeded - 5 warning(s)


** (valac:52363): CRITICAL **: 22:53:14.847: vala_data_type_get_actual_type: assertion 'self != NULL' failed
/home/martin/rpmbuild/BUILD/diorite-4.10.0/src/glib/Dbus.vala:80.2-80.22: warning: unreachable code detected
	assert_not_reached();
	^^^^^^^^^^^^^^^^^^^^^
/home/martin/rpmbuild/BUILD/diorite-4.10.0/src/glib/SocketChannel.vala:69.11-71.3: warning: unreachable catch clause detected
/home/martin/rpmbuild/BUILD/diorite-4.10.0/src/glib/Storage.vala:39.2-39.29: warning: Type `GLib.File[]' can not be used for a GLib.Object property
	public GLib.File[] data_dirs
	^^^^^^^^^^^^^^^^^^^^^^^^^^^^

** (valac:52363): CRITICAL **: 22:53:15.431: vala_gir_writer_skip_implicit_params: assertion 'type != NULL' failed

** (valac:52363): CRITICAL **: 22:53:15.431: vala_gir_writer_write_param_or_return: assertion 'type != NULL' failed

** (valac:52363): CRITICAL **: 22:53:15.431: vala_gir_writer_write_implicit_params: assertion 'type != NULL' failed
/home/martin/rpmbuild/BUILD/diorite-4.10.0/src/glib/Dbus.vala:25.1-25.18: warning: Namespace Dbus does not have a GIR namespace and version annotation
namespace Drt.Dbus {
^^^^^^^^^^^^^^^^^^
glib-2.0.vapi:1508.1-1508.14: warning: Namespace GLib does not have a GIR namespace and version annotation
namespace GLib {
^^^^^^^^^^^^^^

I can not go back to a older valac version.

@jiri-janousek jiri-janousek changed the title valac 0.39.7 error: vala_data_type_get_actual_type: assertion 'self != NULL' failed Multiple warnings with valac 0.39.7 Mar 15, 2018
@jiri-janousek jiri-janousek added the type: bug Something is not working as expected or the documentation is incorrect. label Mar 15, 2018
@jiri-janousek
Copy link
Member

Hello @martinkg, thanks for reporting the issue. However, I cannot see any errors but a few warnings and a successful compilation. I'll keep this ticket open, because I should get rid of the warning.

If Diorite still fails to build on Fedora, please create a new ticket, attach a complete build log as a text file. If you applied any patches, attach them too.

github_attach

@jiri-janousek
Copy link
Member

There are tons of warning: Type XXX can not be used for a GLib.Object property.

@jiri-janousek
Copy link
Member

a lot of compilation warnings on Fedora 28 with diorite-4.10.0 and vala-0.40.0
warnings-f28.txt

jiri-janousek added a commit that referenced this issue Mar 21, 2018
Issue: #20

Signed-off-by: Jiří Janoušek <janousek.jiri@gmail.com>
jiri-janousek added a commit that referenced this issue Mar 21, 2018
Issue: #20

Signed-off-by: Jiří Janoušek <janousek.jiri@gmail.com>
jiri-janousek added a commit that referenced this issue Mar 21, 2018
Issue: #20

Signed-off-by: Jiří Janoušek <janousek.jiri@gmail.com>
@jiri-janousek
Copy link
Member

https://github.com/tiliado/diorite/tree/vala0.40 should contain less Valac warnings.

@martinkg
Copy link
Author

compiled with the las git commit ec98ed1
diorite-4.10-warnings-f28.txt

jiri-janousek added a commit that referenced this issue Mar 23, 2018
Issue: #20

Signed-off-by: Jiří Janoušek <janousek.jiri@gmail.com>
jiri-janousek added a commit to tiliado/nuvolaplayer that referenced this issue Mar 23, 2018
Issue: tiliado/diorite#20

Signed-off-by: Jiří Janoušek <janousek.jiri@gmail.com>
@martinkg
Copy link
Author

martinkg commented Mar 23, 2018

looks now much better with 31408fd
diorite-4.10-warnings2-f28.txt

@jiri-janousek
Copy link
Member

There are only some deprecation warnings left, which can be ignored for now.

@jiri-janousek
Copy link
Member

Diorite mater now builds without any warnings with Vala 0.41.91. Moreover, it treats warnings as fatal by default unless you use ./waf configure --no-strict.

@jiri-janousek jiri-janousek added release notes The work on the ticket is ready, but needs to be released and announced. and removed status: in progress labels Aug 12, 2018
@jiri-janousek
Copy link
Member

@jiri-janousek
Copy link
Member

Released in Nuvola 4.13.0.

@ghost ghost removed the release notes The work on the ticket is ready, but needs to be released and announced. label Oct 15, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Something is not working as expected or the documentation is incorrect.
Projects
None yet
Development

No branches or pull requests

2 participants