[SRU] typo in completion for openssl x509

Bug #690644 reported by Marian Sigler
16
This bug affects 1 person
Affects Status Importance Assigned to Milestone
bash-completion (Debian)
Fix Released
Unknown
bash-completion (Ubuntu)
Fix Released
Low
Unassigned
Maverick
Fix Released
Low
Unassigned

Bug Description

Binary package hint: bash-completion

TEST CASE:

If you write openssl x509 and press TAB there is `-subject-hash` but must be `-subject_hash`.

Regression:

I think there is no options for regression, I only modify specific files in contrib/ dir.

Revision history for this message
Marian Sigler (maix42) wrote :
tags: added: patch
Angel Abad (angelabad)
Changed in bash-completion (Ubuntu):
status: New → Confirmed
importance: Undecided → Low
Angel Abad (angelabad)
Changed in bash-completion (Ubuntu):
assignee: nobody → Angel Abad (angelabad)
status: Confirmed → In Progress
Angel Abad (angelabad)
Changed in bash-completion (Ubuntu):
status: In Progress → Confirmed
assignee: Angel Abad (angelabad) → nobody
Revision history for this message
Benjamin Drung (bdrung) wrote :

Angel, the patch author is wrong. Marian wrote the patch, you applied the patch and added the DEP3 header.

Revision history for this message
Angel Abad (angelabad) wrote : Re: [Bug 690644] Re: typo in completion for openssl x509

Opps! Sorry, you're right, I will upload new debdiff with this issue solved.

Thanks!

Revision history for this message
Angel Abad (angelabad) wrote : Re: typo in completion for openssl x509

Re-upload debdiff with patch author fixed.

Revision history for this message
Artur Rona (ari-tczew) wrote :

In debian/changelog rather use:

  [ Marian Sigler ]
  * debian/patches/fix_typo_in_openssl_completion:
    - Fix typo in openssl completion. (LP: #690644)

In patch please use tag 'Author' for Marian and 'From' for you, Angel.

What about forwarding upstream?

Changed in bash-completion (Ubuntu):
status: Confirmed → Incomplete
Revision history for this message
Artur Rona (ari-tczew) wrote :

OK, David Paleino uploaded it to git upstream. Please add tag:
Origin: upstream, http://git.debian.org/?p=bash-completion/bash-completion.git;a=commit;h=5f04a30f7ab561dc99054445a7f635f3df3a0747

Revision history for this message
Benjamin Drung (bdrung) wrote :

Where is the debdiff? The original debian/changelog was ok.

Revision history for this message
Angel Abad (angelabad) wrote :
Revision history for this message
Angel Abad (angelabad) wrote :

Re-upload debdiff with all issues fixed.

Thanks!

Changed in bash-completion (Ubuntu):
status: Incomplete → Confirmed
Changed in bash-completion (Debian):
status: Unknown → Confirmed
Revision history for this message
Angel Abad (angelabad) wrote :

@Benjamin: The debdiff is uploaded, I applied the Artur's suggested change in debian/changelog, should I revert this change?

Thanks for your advices!

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package bash-completion - 1:1.2-3ubuntu3

---------------
bash-completion (1:1.2-3ubuntu3) natty; urgency=low

  [ Marian Sigler ]
  * debian/patches/fix_typo_in_openssl_completion.patch:
    - Fix typo in openssl completion (LP: #690644)
 -- Angel Abad <email address hidden> Mon, 10 Jan 2011 20:57:38 +0100

Changed in bash-completion (Ubuntu):
status: Confirmed → Fix Released
Revision history for this message
Benjamin Drung (bdrung) wrote :

@Angel: You can keep it. Both are ok.

I uploaded it with one small change: I added ".patch" to the patch filename.

Revision history for this message
Marian Sigler (maix42) wrote :

Woah what a lot of mails ... Thanks :-) (Seeing the debdiff I must say I could have done that myself ;-) )

Angel Abad (angelabad)
description: updated
summary: - typo in completion for openssl x509
+ [SRU] typo in completion for openssl x509
Changed in bash-completion (Ubuntu):
milestone: none → maverick-updates
milestone: maverick-updates → none
Revision history for this message
Angel Abad (angelabad) wrote :

Debdiff for maverick-proposed.

Changed in bash-completion (Ubuntu Maverick):
importance: Undecided → Low
status: New → Triaged
Changed in bash-completion (Debian):
status: Confirmed → Fix Released
Angel Abad (angelabad)
Changed in bash-completion (Ubuntu Maverick):
status: Triaged → Confirmed
Revision history for this message
Kees Cook (kees) wrote :

Thanks for the maverick patch! This looks good; I'll get it uploaded for testing in -proposed.

tags: added: verification-needed
Revision history for this message
Kees Cook (kees) wrote :

This has been uploaded to -proposed. Once it has built, please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Thank you in advance!

Changed in bash-completion (Ubuntu Maverick):
status: Confirmed → Fix Committed
Revision history for this message
Martin Pitt (pitti) wrote : Please test proposed package

Accepted bash-completion into maverick-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

Revision history for this message
Angel Abad (angelabad) wrote :

maverick-proposed package works well for me.

tags: added: verification-done
removed: verification-needed
Revision history for this message
Fabri Velas (fabrivelas) wrote :

The maverick-proposed package now gives the right `-subject_hash` when tab completion is used. Thanks.

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package bash-completion - 1:1.2-2ubuntu1.1

---------------
bash-completion (1:1.2-2ubuntu1.1) maverick-proposed; urgency=low

  * debian/patches/fix_typo_in_openssl_completion:
    - Fix typo in openssl completion (LP: #690644)
  * debian/patches/fix_p4_completion.patch:
    - Fix p4 completion (LP: #652104)
 -- Angel Abad <email address hidden> Mon, 10 Jan 2011 18:07:59 +0100

Changed in bash-completion (Ubuntu Maverick):
status: Fix Committed → Fix Released
tags: added: testcase
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.