install-dependency fails on Ubuntu lucid/10.04

Bug #1272519 reported by J.D.
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Pipelight
Fix Released
Low
Sebastian Lackner

Bug Description

Hi successfully installed pipelight using the prebuilt pluginloader.exe and a custom wine version. However, the script install-dependency uses the program mktemp with the option --suffix, which doesn't exist under lucid, probably it was introduced later. Removing this option (and suffix) works for installing silverlight, I didn't test the other dependencies. It seems the suffix (.exe) is nice but not really necessary. Please fix to support older distributions as well.

pipelight-0.2.4.2

Revision history for this message
Sebastian Lackner (slackner) wrote :

Hi,

thanks for reporting this problem.

The problem is that for newer distros all the "old" command line switches -t and -p are marked as deprecated, which could mean that they'll probably be dropped in a future version. I don't like the idea of downgrading to use deprecated commands - but I agree that a solution / a way which is fully compatible for all distros would be nice.

Unfortunately I haven't any OS with an older version of mktemp right here, so could you please test whats exactly the behaviour for your distro? I would like to see the output of the following commands:

mktemp --suffix ".exe"; echo $?
mktemp -t pipelight.XXXXXXX.exe
mktemp -t pipelight.exe.XXXXXXX

Thanks,
Sebastian

Revision history for this message
J.D. (fungs) wrote :

mktemp --suffix ".exe"; echo $?
mktemp: unrecognized option '--suffix'
Try `mktemp --help' for more information.
1

mktemp -t pipelight.exe.XXXXXXX
/tmp/pipelight.exe.lwarLTW

mktemp -t pipelight.XXXXXXX.exe
mktemp: too few X's in template `pipelight.XXXXXXX.exe'

If you don't rely on any extension, you can simply use mktemp without options. This worked for me!

Changed in pipelight:
assignee: nobody → Sebastian Lackner (slackner)
status: New → In Progress
importance: Undecided → Low
Revision history for this message
Michael Müller (mqchael) wrote :

Hi,

this problem is now fixed by commit https://bitbucket.org/mmueller2012/pipelight/commits/89a9446110702e41ec40de359553379ccf5d2403 though we do not officially support Ubuntu versions which already reached the EOL for the desktop version.

Michael

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