Doesn't build with latest vala 0.15

Bug #917171 reported by Vincent Untz
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Déjà Dup
Fix Released
Low
Unassigned

Bug Description

As I assume vala 0.16 will be the default vala for GNOME 3.4, it'd be nice to have deja-dup build with it too :-)

AssistantOperation.vala:196.21-196.25: error: Argument 1: Cannot pass value to reference or output parameter
      buffer.delete(start, line100);
                    ^^^^^
AssistantOperation.vala:201.24-201.27: error: Argument 1: Cannot pass value to reference or output parameter
    buffer.insert_text(iter, log_line, (int)log_line.length);
                       ^^^^
Compilation failed: 2 error(s), 0 warning(s)

Related branches

Revision history for this message
Vincent Untz (vuntz) wrote :

Here's a rather simple patch to fix this. It's unclear to me if this breaks things with vala 0.14 as I don't have this vala version anymore :/

Revision history for this message
Michael Terry (mterry) wrote :

Ah, the perils of developing with Vala. The patch does break valac-0.14.

My policy has been to use and continue supporting the last cycle's platform (glib/gtk/vala) during the next cycle. This helps any developers that don't want to run inside jhbuild or Ubuntu+1 or whatever.

(I can get away with this policy since I'm not officially a GNOME module. This policy is also a huge time-saver since I'm not on a treadmill of breakages. It would be neat if more official GNOME apps were allowed to do this.)

I would accept a patch that also adds a bit of vapi magic to let this continue working in valac-0.14 (likely just backporting the changed calls from valac-0.16).

Changed in deja-dup:
importance: Undecided → Wishlist
status: New → Triaged
Revision history for this message
Michael Terry (mterry) wrote :

(The policy is also great for testing -- I can have a PPA for oneiric that users can run. Another thing that would help other GNOME apps.)

Revision history for this message
Michael Terry (mterry) wrote :

Fixed. Now DD exposes the version of valac and uses #if statements to allow compilation on valac-0.14 and valac-0.16.

Please report any other breakages if you find them before I do and I'll fix them quicker next time now that I have this framework.

Changed in deja-dup:
importance: Wishlist → Low
milestone: none → 21.5
status: Triaged → Fix Committed
Michael Terry (mterry)
Changed in deja-dup:
milestone: 21.5 → 21.90
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.