Merge version 0.15.10-9 from Debian unstable

Bug #162199 reported by Andrea Gasparini
4
Affects Status Importance Assigned to Milestone
openvrml (Ubuntu)
Fix Released
Wishlist
Unassigned

Bug Description

Merge openvrml 0.15.10-9 from Debian unstable.
Ubuntu component: Universe
Debian component: Main

The difference (apart from changelog) are in debian/control:
 20 -Depends: libopenvrml5c2a (= ${binary:Version}), xlibmesa-gl-dev | libgl-dev, .....
 21 +Depends: libopenvrml5c2a (= ${Source-Version}), libgl1-mesa-dev, ....

The first difference, (binary:Version) is in Debian Changelog:
  7 openvrml (0.15.10-9) unstable; urgency=low
  8
  9 * debian/control:
 10 + Replace ${Source-Version} with ${binary:Version}.

and could be dropped.

The latter isn't reported in Debian Changelog: package xlibmesa-gl-dev exists in ubuntu, but is a transitional package.So we shouldn't depend on it.

Revision history for this message
Andrea Gasparini (gaspa) wrote :

this is a first build of the package with debian control as debian

Changed in openvrml:
assignee: nobody → gaspa
status: New → Confirmed
Changed in openvrml:
assignee: gaspa → norsetto
status: Confirmed → In Progress
Revision history for this message
Cesare Tirabassi (norsetto) wrote :

I still think that we should keep the ubuntu changes, since:

xlibmesa-gl-dev is a transitional package (we should not build-depend or depend on transitional packages)
libgl-dev is a virtual package (provided for instance by libgl1-mesa-dev).

So, the ubuntu changes:

1) Replace xlibmesa-gl-dev | libgl-dev with libgl1-mesa-dev for build-depends
2) Replace xlibmesa-gl-dev | libgl-dev with libgl1-mesa-dev as depends of libopenvrml5-dev

make sense.

Where I agree with you is that these should have indeed been documented in the changelog!

Please resubscribe u-u-s once corrected.

Thanks.

Changed in openvrml:
assignee: norsetto → gaspa
importance: Undecided → Wishlist
Revision history for this message
Andrea Gasparini (gaspa) wrote :
description: updated
Revision history for this message
Andrea Gasparini (gaspa) wrote :

Thanks cesare for your (irc) explanation.
See you at the next report! :D

Revision history for this message
Cesare Tirabassi (norsetto) wrote :

First, next time remember to set the status to confirmed and assign it to nobody, or you risk that no sponsor will look at your bug report!

Concerning your debdiff, you missed the change of the dependencies in your changelog and i think you should change the date to the current one, not keep the MoM/DaD one which is quite old.

Finally, you have added an homepage field to the long description of openvrml-lookat.
Even though this could be in the mom/dad patches, I see no justification of this change in previous changes; beside it seems not very useful to have such a change for one binary package only. Last but not least, it is missing a leading blank space; I would advise you to remove this.

Please attach as well a build log (I cannot build this locally on my machine due to disk trashing).

Once corrected, please resubscribe u-u-s.

Changed in openvrml:
status: In Progress → Invalid
Revision history for this message
Andrea Gasparini (gaspa) wrote :

Ok. I corrected the changelog.

The other errors, as you said, are due from the DaD patch, but i could be more careful, so I removed them.

Revision history for this message
Andrea Gasparini (gaspa) wrote :

Here is the build log

Andrea Gasparini (gaspa)
Changed in openvrml:
assignee: gaspa → nobody
status: Invalid → Confirmed
Revision history for this message
Cesare Tirabassi (norsetto) wrote :

Uploaded (I assume that 0.15.10-10~gaspa2 and 0.15.10-9ubuntu1 are the same sources).
Thanks for your work.

Changed in openvrml:
status: Confirmed → Fix Committed
Revision history for this message
Andrea Gasparini (gaspa) wrote :

Yes, because PPA actually doens't accept the same (or lower) version of a package, and there's no (AFAIK) way to delete them.
thanks, cesare.

Revision history for this message
Cesare Tirabassi (norsetto) wrote :
Changed in openvrml:
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.