Ticket #670 (closed enhancement: fixed)

Opened 6 months ago

Last modified 5 months ago

Erweiterte Optionen für Samba Shares (inkl. Patch)

Reported by: svoop Owned by:
Priority: normal Milestone: freetz-1.2
Component: packages Version: devel
Severity: normal Keywords: samba
Cc: Product ID:
Firmware Version:

Description

Der attachte Patch erweitert das Samba Paket, damit zu einem Samba Share zusätzliche Optionen definiert werden können.

Das ist insbesondere dann sinnvoll, wenn die Festplatte sowohl vis FB/Samba als auch gelegentlich direkt am Computer angeschlossen wird, da es in diesem Fall zu Permissionsproblemen kommen kann. Beispiel:

/var/media/ftp/uStor01 TimeMachine 0 0 force%20user=mac,force%20group=mac,create%20mask=744,directory%20mask=755

Damit wird sichergestellt, dass alle erzeugten Dateien/Dirs die richtigen Permissions erhalten.

Der Patch an sich ist unproblematisch, allerdings müssen all jene, die in "Samba: Freigaben" Shares mit optionalem Kommentar stehen haben, dort vor dem Kommentar ein " - " einsetzen.

Attachments

samba-extra-options.patch Download (2.8 KB) - added by svoop 6 months ago.
samba_no_bash_operators.diff Download (1.7 KB) - added by er13 5 months ago.

Change History

Changed 6 months ago by svoop

  Changed 6 months ago by cinereous

Gehört das nicht eigentlich in die eigenen Freigaben oder die erweiterten Optionen?

  Changed 6 months ago by cinereous

  • product_id 7270 deleted
  • milestone freetz-1.1.3 deleted

  Changed 6 months ago by svoop

Das ist in den eigenen Freigaben, genau dafür ist der Patch ja da :-)

  Changed 5 months ago by oliver

  • status changed from new to accepted
  • milestone set to freetz-1.2

  Changed 5 months ago by oliver

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

(In [4404]) * samba: Allow optional options for shares (by svoop)

  • Perhaps you have to add an "-" before your comments
  • fixes #670

  Changed 5 months ago by oliver

  • status changed from closed to reopened
  • resolution fixed deleted

Die Änderung scheint nicht zu funktionieren. Siehe  IPPF

follow-up: ↓ 8   Changed 5 months ago by SaschaBr

Ich habe mir mal den oben angehängten Patch angeschaut (zugegeben, ich verstehe da nicht allzuviel von), aber kann es sein, das in Zeile 96 vor das "$options" noch ein "options =" hin gehört bzw. das dort fehlt?

in reply to: ↑ 7   Changed 5 months ago by SaschaBr

Replying to SaschaBr:

… kann es sein, das in Zeile 96 vor das "$options" noch ein "options =" hin gehört bzw. das dort fehlt?

Bei genauerer Betrachtung gehört das natürlich so wie es ist, zumindest in dieser Zeile. Dann muss ich leider vorerst passen, sorry.

Changed 5 months ago by er13

  Changed 5 months ago by er13

@SaschaBr: könntest Du bitte den von mir angehängten Patch testen, Danke!

  Changed 5 months ago by cinereous 

if test -d $share

sollten wir nach unseren "Richtlinien" eher mit 

if [ -d "$share" ]; then

schreiben, denke ich.

  Changed 5 months ago by SaschaBr

So, nachdem ich die letzten Tage ein wenig geschwächelt habe (Grippe), habe ich mich heute mal an den Patch gewagt: Also, ich habe frisch ausgecheckt, alles neu im "menuconfig" konfiguriert, obiges Patch eingespielt, Firmware bauen lassen und im Anschluss geflasht. Die merkwürdigen Anzeigen "sh: =~: unknown operand" sind alle verschwunden, und die Freigaben funktionieren (jetzt mit dem zusätzlich "-" vor dem Kommentar in den Freigaben) ohne Probleme.

  Changed 5 months ago by er13

  • status changed from reopened to closed
  • resolution set to fixed

(In [4433]) samba_conf:

  • avoid using bash-specific operators
  • various minor cleanups, fixes & indentation changes
  • fixes #670
Note: See TracTickets for help on using tickets.