Uploaded image for project: 'JDK'
  1. JDK
  2. JDK-7046405

fix for 7028172 causes "sed: command garbled:" message

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Duplicate
    • Icon: P4 P4
    • tbd
    • hs21
    • hotspot
    • None
    • svc
    • b12
    • generic
    • solaris

      Vladimir reports:

      I still see next message after your fix:

      "sed: command garbled: / snv_[0-9][0-9]b"

      I looked and the changes you pushed are different from suggested fix.
      In the suggestion there were double $$:

      + -e '/ snv_[0-9][0-9]$$/b' \

      but in the push only one $:

      + -e '/ snv_[0-9][0-9]$/b' \

      After I added second $ the message is gone:

      make/solaris/makefiles/saproc.make Wed May 18 15:27:42 2011 -0700
      @@ -72,9 +72,9 @@
                 -e '/^[0-4]\. /b' \
                 -e '/^5\.[0-9] /b' \
                 -e '/^5\.10 /b' \
      - -e '/ snv_[0-9][0-9]$/b' \
      - -e '/ snv_[01][0-4][0-9]$/b' \
      - -e '/ snv_15[0-8]$/b' \
      + -e '/ snv_[0-9][0-9]$$/b' \
      + -e '/ snv_[01][0-4][0-9]$$/b' \
      + -e '/ snv_15[0-8]$$/b' \
                 -e 's/.*/-DSOLARIS_11_B159_OR_LATER/' \
                 -e 'p' \
                 )

      so the original fix:

          7028172 3/4 SA needs to adapt to Solaris libproc change made in 6748307

      used single '$' and the follow on fix:

          7043298 4/4 fix for 7028172 causes "Label too long: ..." error message

      fixed the "Label too long:" problem but did not change the
      of single '$' to double '$$'.

            dcubed Daniel Daugherty
            dcubed Daniel Daugherty
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated:
              Resolved:
              Imported:
              Indexed: