Merge lp:~codygarver/wingpanel/packaging-fix-1170094 into lp:~elementary-os/wingpanel/deb-packaging-freya

Proposed by Cody Garver
Status: Merged
Merged at revision: 11
Proposed branch: lp:~codygarver/wingpanel/packaging-fix-1170094
Merge into: lp:~elementary-os/wingpanel/deb-packaging-freya
Diff against target: 26 lines (+7/-1)
2 files modified
debian/changelog (+6/-0)
debian/control (+1/-1)
To merge this branch: bzr merge lp:~codygarver/wingpanel/packaging-fix-1170094
Reviewer Review Type Date Requested Status
Sergey "Shnatsel" Davidoff (community) Approve
Review via email: mp+159486@code.launchpad.net
To post a comment you must log in.
12. By Cody Garver

Fix my email.

Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :

In fact, no need to bump it to 2:.., 1:.. is fine because {debupstream} we use for recipes is not pick up the epoch version.

By the way, I still have to revert 1:.. on many branches...

review: Needs Fixing
Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :

Also, why the valac version bump?

13. By Cody Garver

Don't bump epoch.

Revision history for this message
Cody Garver (codygarver) wrote :

I reduced the epoch to 1 as requested. The valac bump is for consistency with other branches that request that. It's what valac --version prints from valac-0.16 package.

Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :

0.16.0 is the latest version available from Ubuntu repos, so you make the package unbuildable without a valac backport. You'd better have a a good reason for doing that!

review: Needs Information
Revision history for this message
Victor Martinez (victored) wrote :

Hey Cody,

I actually agree with Sergey regarding valac. As far as my knowledge goes, we are currently not depending on any *.vapi file exclusively available to valac 0.16.1, so everything should build fine with vala 0.16.0

Let's bump the version when absolutely necessary. We'd also need to change the version in CMakeLists.txt to request 0.16.0 again.

Please let me know what you think.

14. By Cody Garver

Revert valac version bump.

Revision history for this message
Cody Garver (codygarver) wrote :

Reverted valac version bump.

Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :

Looks good to me now. I'll merge this and change {debupstream} to {debversion} in the recipe.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2013-04-01 18:08:53 +0000
3+++ debian/changelog 2013-04-20 17:35:30 +0000
4@@ -1,3 +1,9 @@
5+wingpanel (1:0.2.2) precise; urgency=low
6+
7+ * New upstream release.
8+
9+ -- Cody Garver <cody@elementaryos.org> Wed, 17 Apr 2013 14:58:56 -0500
10+
11 wingpanel (1:0.1) oneiric; urgency=low
12
13 * Initial Release.
14
15=== modified file 'debian/control'
16--- debian/control 2013-03-27 07:29:32 +0000
17+++ debian/control 2013-04-20 17:35:30 +0000
18@@ -10,7 +10,7 @@
19 libgtk-3-dev,
20 libindicator3-dev (>= 0.3.90),
21 valac-0.16 | valac (>= 0.16)
22-Standards-Version: 3.9.1
23+Standards-Version: 3.9.3
24 Homepage: https://launchpad.net/wingpanel
25 Vcs-Bzr: https://code.launchpad.net/~wingpanel-devs/wingpanel/wingpanel-gtk3
26 Vcs-Browser: http://bazaar.launchpad.net/~wingpanel-devs/wingpanel/wingpanel-gtk3/files

Subscribers

People subscribed via source and target branches