← Back to team overview

hugin-devs team mailing list archive

[Bug 679290] Re: autopano-noop.sh should be named autopano-noop (without .sh)

 

The ".sh" extension is a "style" issue. Some prefer it like that, some
don't. Whomever wrote the code first gets to determine this. Sorry about
that.

With builtin CP detectors the situation is improving. I don't see noop
in my defaults list anymore. Let's try to forget about this for now.

** Changed in: hugin
       Status: New => Fix Released

** Changed in: hugin
   Importance: Undecided => Low

-- 
You received this bug notification because you are a member of Hugin
Developers, which is subscribed to Hugin.
https://bugs.launchpad.net/bugs/679290

Title:
  autopano-noop.sh should be named autopano-noop (without .sh)

Status in Hugin - Panorama Tools GUI:
  Fix Released

Bug description:
  hugin 2009.2.0 svn 4385 uses "autopano-noop.sh" as cp detector by default. There are a couple of issues with this:

* autopano-noop.sh is not installed. So although I have got a working autopano-sift-c and am upgrading from hugin 0.8.0 which happily used this I end up with an error message in 2009.2. The only way to fix this to either manually install autopano-noop.shor to edit the preferences and set autopano-sift-c instead of autopano-noop.sh.

* The file should not have a .sh extension. I think it is not too far fetched that perhaps the file might end up as something that is not a shellscript in the future. If this happens you either need to update the defaults again or end up with a non-shell-script with a .sh extension.

* Actually it would be very nice if if the wrapper script worked with both autopano-sift and autopano-sift-c.

Thank's, cu andreas