Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ftp permission #1708

Closed
mc-butler opened this issue Oct 12, 2009 · 12 comments
Closed

ftp permission #1708

mc-butler opened this issue Oct 12, 2009 · 12 comments
Assignees
Labels
area: vfs Virtual File System support prio: medium Has the potential to affect progress ver: 4.7.0-pre3 Reproducible in version 4.7.0-pre3
Milestone

Comments

@mc-butler
Copy link

Important

This issue was migrated from Trac:

Origin https://midnight-commander.org/ticket/1708
Reporter x905

then i copy file to ftp - mc still try to set permissions because "preserve Attributes" set (as default) - why ?

@mc-butler
Copy link
Author

Changed by zaytsev (@zyv) on Oct 12, 2009 at 13:14 UTC (comment 1)

And why wouldn't it?

@mc-butler
Copy link
Author

Changed by x905 on Oct 13, 2009 at 4:30 UTC (comment 2)

because ftp dont have attributes
in previous builds (for example "4.6.3-[529]") mc works as expected - dont try to set unsettable attribute

@mc-butler
Copy link
Author

Changed by x905 on Nov 5, 2009 at 6:27 UTC (comment 3)

please accept this ticket - its very annoying bug

@mc-butler
Copy link
Author

Changed by slavazanko (@slavaz) on Nov 5, 2009 at 10:50 UTC (comment 4)

  • Owner set to slavazanko
  • Status changed from new to accepted

@mc-butler
Copy link
Author

Changed by slavazanko (@slavaz) on Nov 5, 2009 at 11:02 UTC (comment 5)

  • Milestone changed from 4.7 to 4.7.0
  • Severity changed from no branch to on review
  • Component changed from mc-core to mc-vfs

Created branch 1708_ftp_permission

Initial [74815e7a2210a4ebe572f15d84def3b0ec60986d]

Review, please.

@mc-butler
Copy link
Author

Changed by x905 on Nov 5, 2009 at 11:53 UTC (comment 6)

works ok

@mc-butler
Copy link
Author

Changed by zaytsev (@zyv) on Nov 9, 2009 at 18:15 UTC (comment 7)

Ah, I see what you meant to say now. Actually it should, since FTP DOES have attributes, but it shouldn't warn you about errors if ignore_ftp_chattr_errors is set.

This change obsoletes 02_ignore_ftp_chmod_error.patch , just a note to myself.

@mc-butler
Copy link
Author

Changed by iNode (@iNode) on Nov 11, 2009 at 6:22 UTC (comment 8)

  • Votes set to iNode

@mc-butler
Copy link
Author

Changed by metux (@metux) on Nov 11, 2009 at 16:50 UTC (comment 9)

  • Votes changed from iNode to iNode metux

we probably need some generic solution.

a similar problem happens when copying files to a fat-fs.

@mc-butler
Copy link
Author

Changed by metux (@metux) on Nov 11, 2009 at 16:50 UTC (comment 10)

  • Severity changed from on review to approved

@mc-butler
Copy link
Author

Changed by slavazanko (@slavaz) on Nov 12, 2009 at 9:03 UTC (comment 11)

  • Resolution set to fixed
  • Status changed from accepted to testing
  • Votes changed from iNode metux to commited-master
  • Severity changed from approved to merged

merge [bbb0250]

@mc-butler
Copy link
Author

Changed by slavazanko (@slavaz) on Nov 12, 2009 at 9:03 UTC (comment 12)

  • Status changed from testing to closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: vfs Virtual File System support prio: medium Has the potential to affect progress ver: 4.7.0-pre3 Reproducible in version 4.7.0-pre3
Development

No branches or pull requests

2 participants