Bug 261456 - textproc/xmlcatmgr: DEINSTALL script failed
Summary: textproc/xmlcatmgr: DEINSTALL script failed
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Hiroki Sato
URL:
Keywords:
: 262861 (view as bug list)
Depends on:
Blocks:
 
Reported: 2022-01-25 05:36 UTC by takefu
Modified: 2022-07-20 00:22 UTC (History)
5 users (show)

See Also:


Attachments
xmlcatmgr-2.2_2.patch (2.19 KB, patch)
2022-01-25 05:36 UTC, takefu
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description takefu 2022-01-25 05:36:18 UTC
Created attachment 231296 [details]
xmlcatmgr-2.2_2.patch

fix:
  DEINSTALL script defect correction.
  PORTCLIPPY(1) compliant.


[***/***] Deinstalling xmlcatmgr-2.2_2...
Invalid skip1: ]
usage: cmp [-l | -s | -x] [-hz] file1 file2 [skip1 [skip2]]
Invalid skip1: ]
usage: cmp [-l | -s | -x] [-hz] file1 file2 [skip1 [skip2]]
Invalid skip1: ]
usage: cmp [-l | -s | -x] [-hz] file1 file2 [skip1 [skip2]]
Invalid skip1: ]
usage: cmp [-l | -s | -x] [-hz] file1 file2 [skip1 [skip2]]
Comment 1 Alexander Vereeken 2022-03-20 15:31:56 UTC
A friendly ping :)
Comment 2 Alexander Pravkin 2022-04-14 16:00:19 UTC
(In reply to takefu from comment #0)

Looks like there's unnecessary closing bracket in files/pkg-deinstall.in script:

if cmp -s ${T} ${CATALOG_MASTER_SGML} ]; then

(and so on 3 more times)
Comment 3 takefu 2022-06-01 03:19:07 UTC
(In reply to Alexander Pravkin from comment #2)


HRS@FreeBSD.org wants to be busy.
I can't commit yet :-)
Comment 5 Alan Somers freebsd_committer freebsd_triage 2022-07-18 14:48:20 UTC
*** Bug 262861 has been marked as a duplicate of this bug. ***
Comment 6 takefu 2022-07-19 05:52:48 UTC
(In reply to Alan Somers from comment #4)

Thank you for your commitment.
However,
PORTCLIPPY(1) Compliance?
Comment 7 Alan Somers freebsd_committer freebsd_triage 2022-07-19 12:55:54 UTC
(In reply to takefu from comment #6)
Oh, the variable ordering?  I think the committer found this issue independently, not through bugzilla.  That's why he didn't commit that part of your patch.
Comment 8 takefu 2022-07-20 00:22:23 UTC
(In reply to Alan Somers from comment #7)

I see.
It didn't mean that the committer was busy. :-)