prevu crashes on `prevu libxine1`

Bug #95755 reported by Andrew Ash
2
Affects Status Importance Assigned to Milestone
Prevu
Fix Released
Low
John Dong

Bug Description

I'm trying to backport xine 1.1.4 from feisty to edgy on my edgy machine, and had prevu crash. I understand that backporting xine may be a very difficult thing to do, since it's used in so many places, but I think that a crash in prevu shouldn't be warranted.

I followed http://ubuntuforums.org/showthread.php?t=268687 to install prevu and use it.

$ apt-cache policy prevu
prevu:
  Installed: 1:0.4.1-0ubuntu1
  Candidate: 1:0.4.1-0ubuntu1
  Version table:
 *** 1:0.4.1-0ubuntu1 0
        100 /var/lib/dpkg/status

I'm attaching the output of `prevu libxine1` to illustrate the crash.

Revision history for this message
Andrew Ash (ash211) wrote :

Traceback (most recent call last):
  File "/usr/bin/prevu", line 146, in ?
    BackportFromAPT(sys.argv[1],DIST).backport()
  File "/usr/bin/prevu", line 86, in backport
    self.do_compile()
  File "/usr/bin/prevu", line 67, in do_compile
    raise ValueError("Build failed.")
ValueError: Build failed.

Revision history for this message
John Dong (jdong) wrote :

This is not a bug in prevu. If you look above, you will see the "real" error message from thebuild process (most likely a dependency that can't be satisfied)

Prevu "crashing" (printing a stackdump) is just the way prevu reports that a build fails.

Changed in prevu:
status: Unconfirmed → Rejected
Revision history for this message
Andrew Ash (ash211) wrote :

I see what was going on. It needed libpulse-dev.

Still, I think catching the exception and printing a message would be more user-friendly. Something maybe along the lines of "Prevu failed to create a package of _____. Check the build output above for clues to why it failed." Thanks

Changed in prevu:
status: Rejected → Unconfirmed
Revision history for this message
John Dong (jdong) wrote :

I agree, I'll get it to print out some friendlier error messages.

To make matters "worse", apport in Feisty is probably catching all these exceptions and annoying the crap out of users *shudder*

Changed in prevu:
assignee: nobody → jdong
importance: Undecided → Low
status: Unconfirmed → Confirmed
Revision history for this message
John Dong (jdong) wrote :

Initial fix is in this bundle; will commit once I get home.

Changed in prevu:
status: Confirmed → In Progress
Revision history for this message
John Dong (jdong) wrote :

Clearer error messages have been committed to the latest prevu.

Changed in prevu:
status: In Progress → 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.