[riot-notifications] [RIOT-OS/RIOT] Small fatfs usability fixes (#16800)

chrysn notifications at github.com
Thu Sep 2 17:57:59 CEST 2021


### Contribution description

This is a collection of the small things that make life easier for someone who wants to give fatfs a try:

* Name the tarball of tests/pkg_fatfs .tar.bz2 (tools like [atool](https://www.nongnu.org/atool/) do look at extensions)
* Make the extraction routine work even when bin/ has not yet been created (with Makefile style fixes as bycatch)
* **Tell user what to do when enabling fatfs to get a usable example (because the disk can't format itself)**

### Testing procedure

* In `tests/pkg_fatfs`, run (there is not automation for this, is there?):

    ```
    $ make image all term
    > mount 0
    mounting file system image...
    [OK]
    Volume name: 
    0,124 GiB of 0,124 GiB available
    > ls
    //TEST.TXT
    ```

* In `examples/filesystem`, follow the new set of instructions and observe that indeed the file system becomes usable with a TEST.TXT present from startup.
You can view, comment on, or merge this pull request online at:

  https://github.com/RIOT-OS/RIOT/pull/16800

-- Commit Summary --

  * tests/pkg_fatfs: Set tarball extension to match content
  * tests/pkg_fatfs: Use ${Q} instead of @
  * tests/pkg_fatfs: Create required bin/ directory
  * examples/filesystem: Explain how to use fatfs

-- File Changes --

    M examples/filesystem/README.md (44)
    M tests/pkg_fatfs/Makefile (5)
    M tests/pkg_fatfs/create_fat_image_file.sh (2)
    R tests/pkg_fatfs/riot_fatfs_disk.tar.bz2 (0)

-- Patch Links --

https://github.com/RIOT-OS/RIOT/pull/16800.patch
https://github.com/RIOT-OS/RIOT/pull/16800.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/16800
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210902/708764e2/attachment-0001.htm>


More information about the notifications mailing list