[wp-meta] [Making WordPress.org] #1570: Reviewer Admin

[Making WordPress.org]

#1570: Reviewer Admin
------------------------------+---------------------------------------
 Reporter:  obenland          |       Owner:
     Type:  enhancement       |      Status:  new
 Priority:  normal            |   Milestone:  Plugin Directory v3 - M1
Component:  Plugin Directory  |  Resolution:
 Keywords:                    |
------------------------------+---------------------------------------

Comment (by dd32):

 Replying to [comment:77 obenland]:
 > Replying to [comment:76 dd32]:
 > > @obenland You should use the `-delete` flag here instead of `-exec rm`
 :)
 >
 > Thanks for taking a look! I tried, but for some reason it doesn't
 actually remove it. I tried:
 > {{{
 > #!bash
 > exec( "find {$esc_directory} -name '__MACOSX' -delete" );
 > }}}
 >
 > Am I missing something?

 Doesn't seem like it, It looks like on linux it doesn't remove non-empty
 directories
 This should work, but I don't think it needs changing, I'll take my
 recommendation back :)
 {{{
 #!bash
 exec( "find {$esc_directory} -path '*/__MACOSX*' -delete" );
 }}}

 You might also want to look for `.DS_Store` files from windows-based
 plugin authors.

--
Ticket URL: <https://meta.trac.wordpress.org/ticket/1570#comment:78>
Making WordPress.org <https://meta.trac.wordpress.org/>
Making WordPress.org