Ticket #1450 (closed defect: fixed)

Opened 13 months ago

Last modified 2 months ago

FISH: timestamps don't show correctly

Reported by: slavazanko Owned by: angel_il
Priority: major Milestone:
Component: mc-vfs Version: master
Severity: no branch Keywords:
Cc: bonoba@…, isee@…, imdagger@…, lly, zyv Votes for changeset:
Blocking: Blocked By: #2242

Description

Test case:
1) Make FISH-connect to some host;
2) sort files by modification time;
3) will see older files (from previous year) as new files

For check, connect to same host and enter in command line:

ls -lt

Thanks for bugreport to Alexander Glyzov.

Attachments

fish_vfs_date.patch (0.7 kB) - added by IMDagger 10 months ago.
Fix for FISHFS files modification date

Change History

Changed 13 months ago by smart

  • cc isee@… added

Changed 10 months ago by IMDagger

Fix for FISHFS files modification date

Changed 10 months ago by IMDagger

  • cc imdagger@… added

Because first part of FISH-ls script is Perl script. But localtime function returns dates like:
Fri Sep 18 10:40:28 2009
But in utilvfs.c is written: So both year and time is not allowed. fish_vfs_date.patch fixed write format of date.

Changed 10 months ago by slavazanko

  • owner set to slavazanko
  • status changed from new to accepted

Changed 10 months ago by slavazanko

  • severity changed from no branch to on review

Created branch 1450_fish_timestamps

Changesets:

Review, please.

Changed 10 months ago by iNode

  • votes set to iNode

Changed 10 months ago by slavazanko

  • milestone changed from 4.7 to 4.7.0

Changed 9 months ago by andrew_b

  • votes changed from iNode to iNode andrew_b
  • severity changed from on review to approved

Changed 9 months ago by slavazanko

  • status changed from accepted to testing
  • votes changed from iNode andrew_b to commited-master
  • resolution set to fixed
  • severity changed from approved to merged

Changed 9 months ago by slavazanko

  • status changed from testing to closed

Changed 9 months ago by angel_il

  • status changed from closed to reopened
  • votes commited-master deleted
  • resolution fixed deleted
  • severity changed from merged to no branch

commit: 4fc08a729e1aefe22f1965e2536a88bdfcc0cf62 broke the FISH under my WL-500gP device (oleg's firmware)

all catalogs and files has ' n3' postfix, like this:

│/dev n3                  │ 
│/home n3                 │ 
│/lib n3                  │ 
│/mnt n3                  │ 
│/opt n3                  │ 
│/proc n3                 │ 
│/sbin n3                 │ 
│/tmp n3                  │ 
│/usr n3                  │ 
│/www n3                  │ 
│!etc                     │ 
│*linuxrc n3              │ 
│!var                     │ 
│ .version n3               

Changed 9 months ago by angel_il

/bin/ls -lan /
-rw-r--r--    1 0        0              10 Feb 28  2009 .version
drwxr-xr-x    1 0        0             360 Feb 28  2009 bin
drwxr-xr-x    1 0        0               0 Jan  1  1970 dev
lrwxrwxrwx    1 0        0               7 Feb 28  2009 etc -> tmp/etc
drwxr-xr-x    1 0        0               0 Feb 28  2009 home
drwxr-xr-x    1 0        0             164 Feb 28  2009 lib
-rwxr-xr-x    1 0        0            2032 Feb 28  2009 linuxrc
drwxr-xr-x    1 0        0               0 Feb 28  2009 mnt
drwxr-xr-x   17 0        0            4096 Mar 15  2009 opt
dr-xr-xr-x   34 0        0               0 Jan  1  2000 proc
drwxr-xr-x    1 0        0             522 Feb 28  2009 sbin
drwxrwxrwx    1 0        0               0 Jan  1  2000 tmp
drwxr-xr-x    1 0        0              71 Feb 28  2009 usr
lrwxrwxrwx    1 0        0               7 Feb 28  2009 var -> tmp/var
drwxr-xr-x    1 0        0            2229 Feb 28  2009 www
ls -lan /
total 15
drwxr-xr-x  1 0 0  360 2009-02-28 21:03 bin
drwxr-xr-x  1 0 0    0 1970-01-01 03:00 dev
lrwxrwxrwx  1 0 0    7 2009-02-28 21:04 etc -> tmp/etc
drwxr-xr-x  1 0 0    0 2009-02-28 21:04 home
drwxr-xr-x  1 0 0  164 2009-02-28 21:04 lib
-rwxr-xr-x  1 0 0 2032 2009-02-28 21:03 linuxrc
drwxr-xr-x  1 0 0    0 2009-02-28 21:04 mnt
drwxr-xr-x 17 0 0 4096 2009-03-15 00:09 opt
dr-xr-xr-x 34 0 0    0 2000-01-01 03:00 proc
drwxr-xr-x  1 0 0  522 2009-02-28 21:03 sbin
drwxrwxrwx  1 0 0    0 2000-01-01 03:00 tmp
drwxr-xr-x  1 0 0   71 2009-02-28 21:04 usr
lrwxrwxrwx  1 0 0    7 2009-02-28 21:04 var -> tmp/var
-rw-r--r--  1 0 0   10 2009-02-28 21:03 .version
drwxr-xr-x  1 0 0 2229 2009-02-28 21:03 www

Changed 9 months ago by slavazanko

  • severity changed from no branch to on review

Created branch 1450_fish_dir_load_fix

Initial changeset: a3ee5860bbf78cd4f65dd12eebba46397800b02d

Review

Changed 9 months ago by andrew_b

  • votes set to andrew_b

Changed 9 months ago by angel_il

  • votes changed from andrew_b to andrew_b angel_il
  • severity changed from on review to approved

Changed 9 months ago by slavazanko

  • status changed from reopened to closed
  • votes changed from andrew_b angel_il to commited-master
  • resolution set to fixed
  • severity changed from approved to merged

Changed 4 months ago by lly

  • status changed from closed to reopened
  • version changed from 4.7.0-pre1 to 4.7.0.5
  • resolution fixed deleted

Unfortunately, both 4.7.0.5 & 4.7.2 mc versions has broken FISH on embedded device with busybox 1.16.1 shell - all catalog & files looks like this:

┌<─ /#sh:admin@192.168.1.100/tmp ─.[^]>┐
│'n     Name      │ Size  │Modify time │
│/..              │UP--DIR│May  6 19:32│
│/."              │    240│Jan  1 03:00│
│/.."             │    170│May  5 00:00│
│/etc"            │    460│Jan  1 03:00│
│/local"          │    140│Jan  1  2000│
│/mnt"            │     40│Jan  1 03:00│
│/ppp"            │    100│Jan  1 03:00│
│/var"            │    180│Jan  1 03:00│
│ filter_rules"   │   1310│Jan  1 03:00│
│ nat_rules"      │    290│Jan  1 03:00│
│ resolv.conf"    │      0│Jan  1 03:00│
│ syslog.log"     │   5121│Jan  1 03:07│

i.e. with extra quote after name. Native ls output on device itself:

$ ls -lan /tmp
drwxrwxrwt    7 0        0              240 Jan  1 03:00 .
drwxr-xr-x   14 0        0              170 May  5  2010 ..
drwxr-xr-x    4 0        0              460 Jan  1 03:00 etc
-rw-r--r--    1 0        0             1310 Jan  1 03:00 filter_rules
drwxr-xr-x    5 0        0              140 Jan  1  2000 local
drwxr-xr-x    2 0        0               40 Jan  1 03:00 mnt
-rw-r--r--    1 0        0              290 Jan  1 03:00 nat_rules
drwxr-xr-x    2 0        0              100 Jan  1 03:00 ppp
-rw-r--r--    1 0        0                0 Jan  1 03:00 resolv.conf
-rw-r--r--    1 0        0             5954 Jan  1 03:15 syslog.log
drwxr-xr-x    9 0        0              180 Jan  1 03:00 var

Changed 4 months ago by lly

Sorry, forgot to mention that bug is observed on 4.7.0.1,4.7.0.2,4.7.0.3,4.7.0.4 too and absent in 4.7.0-pre4.

Changed 3 months ago by angel_il

branch: 1450_fish_fix
changeset: 0bbab2338ea02b29ffa2ee9655a6ea19634d527c

Changed 3 months ago by angel_il

  • cc lly, added
  • owner changed from slavazanko to angel_il
  • status changed from reopened to accepted

Changed 3 months ago by angel_il

  • votes committed-master deleted
  • severity changed from merged to on review

Changed 3 months ago by andrew_b

  • keywords FISH removed
  • votes set to andrew_b

Changed 3 months ago by slavazanko

  • votes changed from andrew_b to andrew_b slavazanko
  • severity changed from on review to approved

Changed 3 months ago by angel_il

  • votes andrew_b slavazanko deleted

Changed 3 months ago by andrew_b

  • severity changed from approved to on rework

Changed 3 months ago by andrew_b

  • milestone changed from 4.7.0 to 4.7

Changed 2 months ago by lly

Clarification - problem described in comment 16, appeared with recent busybox versions which supports "ls -Qlan" only.

Changed 2 months ago by zyv

  • cc zyv added; removed
  • version changed from 4.7.0.5 to master

Changed 2 months ago by zyv

  • blockedby 2242 added

Changed 2 months ago by lly

in 4.7.3 release my problem (see comment 16) was gone.

Changed 2 months ago by zyv

  • status changed from accepted to testing
  • resolution set to fixed

Changed 2 months ago by zyv

  • status changed from testing to closed
  • severity changed from on rework to no branch
  • milestone 4.7 deleted
Note: See TracTickets for help on using tickets.