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

compilation warning messages #14

Open
fossfreedom opened this issue Mar 14, 2018 · 2 comments
Open

compilation warning messages #14

fossfreedom opened this issue Mar 14, 2018 · 2 comments

Comments

@fossfreedom
Copy link

During compilation see the following warnings - something to worry about?

../src/BudgieWeatherApplet.vala:45.9-45.34: warning: Gtk.Widget.margin_left has been deprecated since 3.12
../src/BudgieWeatherApplet.vala:45.38-45.64: warning: Gtk.Widget.margin_right has been deprecated since 3.12
../src/BudgieWeatherApplet.vala:49.9-49.29: warning: Gtk.Widget.margin_left has been deprecated since 3.12
../src/BudgieWeatherApplet.vala:49.33-49.59: warning: Gtk.Widget.margin_right has been deprecated since 3.12
../src/BudgieWeatherApplet.vala:66.36-66.39: warning: Argument 1: Cannot pass null to non-null parameter type
        new GWeather.LocationEntry(null);
                                   ^^^^

build log https://launchpadlibrarian.net/360636309/buildlog_ubuntu-bionic-i386.budgie-weather-applet_0.1-0ubuntu2~bionic+build3_BUILDING.txt.gz

ubuntu budgie 18.04 building against GNOME 3.28 gweather library

@ilgarmehmetali
Copy link
Owner

I cant exactly remember what but i had some problems with margins and i had to use deprecated properties. Should try the new properties and see if i would still face with problems.

Last warning is because of a little hack and is related to this:

// to solve template giving "Invalid object type 'GWeatherLocationEntry'" error"
new GWeather.LocationEntry(null);

None of them should cause any trouble for the time being but it would be better to find better solutions.

@fossfreedom
Copy link
Author

ok - I can do a PR with the new properties and an alternative solution for the "null" hack.

However concerned that the applet update button is behaving very intermittently - see the issue about the compilation problems

fossfreedom added a commit to UbuntuBudgie/budgie-weather-applet that referenced this issue Mar 14, 2018
fossfreedom added a commit to UbuntuBudgie/budgie-weather-applet that referenced this issue Mar 14, 2018
fossfreedom added a commit to UbuntuBudgie/budgie-weather-applet that referenced this issue Mar 14, 2018
 GTK has been bumped due to removing deprecated properties ilgarmehmetali#14
 GWeather has been bumped due to API changing in GNOME 3.28 ilgarmehmetali#13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants