Does not work because file command changed its output

Bug #998610 reported by André
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
sqldeveloper-package (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

Please, change line 381 of the make-sqldeveloper-package script from this:

 # Set executable bit
 ${FIND} "${OPTDIR}" ! \( -type d -o -name "*.jar" \) |${XARGS} ${XARGS_OPTS} ${FILE} ${FILE_OPTS} |${GREP} ${GREP_OPTS} "shell script text executable" |${CUT} ${CUT_OPTS_FUNC_CLEAN} |${XARGS} ${XARGS_OPTS} ${CHMOD} ${CHMOD_OPTS}

To this:

 ${FIND} "${OPTDIR}" ! \( -type d -o -name "*.jar" \) |${XARGS} ${XARGS_OPTS} ${FILE} ${FILE_OPTS} |${GREP} ${GREP_OPTS} "shell script" |${GREP} ${GREP_OPTS} "text executable" |${CUT} ${CUT_OPTS_FUNC_CLEAN} |${XARGS} ${XARGS_OPTS} ${CHMOD} ${CHMOD_OPTS}

Reason:

file command version 5.09-2 does not report shell scripts the way the above script is waiting. It reports like "POSIX shell script, ASCII text executable" or "Bourne-again shell script, ASCII text executable". So I splitted the one grep command into two grep commands.

Extra-info:

$ lsb_release -rd
Description: Ubuntu 12.04 LTS
Release: 12.04
$ apt-cache policy sqldeveloper-package
sqldeveloper-package:
  Instalado: 0.2.3+nmu1
  Candidato: 0.2.3+nmu1
  Tabela de versão:
 *** 0.2.3+nmu1 0
        500 http://br.archive.ubuntu.com/ubuntu/ precise/multiverse i386 Packages
        100 /var/lib/dpkg/status

Tags: patch
Revision history for this message
André (aedgsilva) wrote :
Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "bug#998610.patch" of this bug report has been identified as being a patch. The ubuntu-reviewers team has been subscribed to the bug report so that they can review the patch. In the event that this is in fact not a patch you can resolve this situation by removing the tag 'patch' from the bug report and editing the attachment so that it is not flagged as a patch. Additionally, if you are member of the ubuntu-reviewers team please also unsubscribe the team from this bug report.

[This is an automated message performed by a Launchpad user owned by Brian Murray. Please contact him regarding any issues with the action taken in this bug report.]

tags: added: patch
Revision history for this message
Salvatore Bonaccorso (carnil) wrote :
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package sqldeveloper-package - 0.2.4

---------------
sqldeveloper-package (0.2.4) unstable; urgency=high

  * Addressed bugs for inclusion in Wheezy's freeze:
    - Replaced dos2unix with tofrodos (LP: #560803,#626272) (Closes: #568982)
      (Reported by Seth Rosenblum)
    - Split grep for shell script (LP: #985810,#998610) (Closes: #692534)
      (Reported by Bruno Medeiros and patch by Brad Powell)
    - Download links updated (Closes: #618650)
      (Reported by Sergio Fernandez)
  * Fixed JDK dependencies for pre and post SQL Developer v2
  * Updated documentation regarding above changes
  * Updated OTN license information to current version
  * Tested against v2.x and v3.x release families of SQL Developer
  * Bumped version to reflect functionality

 -- Lazarus Long <email address hidden> Wed, 14 Nov 2012 11:05:12 +0000

Changed in sqldeveloper-package (Ubuntu):
status: New → 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.