Ticket #1477 (closed defect: fixed)

Opened 9 years ago

Last modified 9 years ago

incorrect detection of compressed patchfs

Reported by: dborca Owned by: slavazanko
Priority: major Milestone: 4.7.0-pre2
Component: mc-vfs Version: 4.7.0-pre1
Keywords: patchfs Cc: dborca@…
Blocked By: Blocking:
Branch state: Votes for changeset: committed-master

Description

patchfs.in incorrectly detects compressed patchfiles, based on filename. My file-4.21 prints the following output:
mc-4.7.0-pre1.lzma_tar.patch.gz: gzip compressed data, was [...]
and then myin/myout subroutines match /lzma/

Attached patch (against 4.7.0-pre1) fixes this (supposely).

Attachments

mc-4.7.0-pre1.patchfs.patch.gz (506 bytes) - added by dborca 9 years ago.
mc-4.7.0-pre1.patchfs.patch (1.4 KB) - added by angel_il 9 years ago.
lzma.patch (9.6 KB) - added by angel_il 9 years ago.
patch by Reynir Stefansson <reynirhs mi is>

Change History

Changed 9 years ago by dborca

comment:1 Changed 9 years ago by dborca

  • Component changed from mc-core to mc-vfs

comment:2 Changed 9 years ago by dborca

  • Cc dborca@… added

comment:3 Changed 9 years ago by slavazanko

stay in pre2

Changed 9 years ago by angel_il

comment:4 Changed 9 years ago by angel_il

this patch don't work for me

Changed 9 years ago by angel_il

patch by Reynir Stefansson <reynirhs mi is>

comment:5 Changed 9 years ago by slavazanko

  • Status changed from new to accepted
  • Owner set to slavazanko
  • severity changed from no branch to on review

created branch 477_incorrect_detection_compressed_patchfs (parent: master)

initial changeset:fbe8eb8f2a7f01bb9b8aafdc02f021aefb1b9c00

Review, please.

About lzma.patch:

  1. patch not related to this ticket and
  2. patch partially applied to master.

If someone wants - please, open new ticket with proper patch.

comment:6 Changed 9 years ago by slavazanko

branch rebased. initial changeset:ad42905ecf5498fbf132fa862ae4386a5f0b7739

I forgot to add real author of patch, sorry.

comment:7 Changed 9 years ago by angel_il

  • Votes for changeset set to angel_il

comment:8 Changed 9 years ago by slavazanko

  • Votes for changeset changed from angel_il to angel_il slavazanko
  • severity changed from on review to approved

comment:9 Changed 9 years ago by slavazanko

  • Status changed from accepted to testing
  • Votes for changeset changed from angel_il slavazanko to commited-master
  • Resolution set to fixed
  • severity changed from approved to merged

comment:10 Changed 9 years ago by slavazanko

  • Status changed from testing to closed
Note: See TracTickets for help on using tickets.