Tutti i miei sbagli, versione 7 Marzo 2012 al XPUG mi

Tutti i miei sbagli Gli errori di un Wannabe Open Source Developer

Andrea Francia

Thursday, March 8, 12

Contenuti

Il progettoGli Sbagli• Generic Naming Issue• Using Bash for functional

testing• Feedback• License• Project hostingUna cosa giustaConclusioniApplausi


trash-cli$ trash foo

$ list-trash

2012-03-02 19:43:44 /Users/andrea/foo

$ touch foo; trash foo

$ list-trash

2012-03-02 19:43:44 /Users/andrea/foo

2012-03-02 19:44:04 /Users/andrea/foo

$ restore-trash

0 2012-03-02 19:43:44 /Users/andrea/foo

1 2012-03-02 19:44:04 /Users/andrea/foo

What file to restore [0..1]:0


Generic Naming Issue


Comandi del pacchettotrash Butta un file

empty-trash Svuota tutto il cestino

list-trash Elenca il contenuto

restore-trash Recupera un file


Debian


Fedora


trashlist-trashrestore-trashempty-trash

Debian

FedoraQArejected

OK


Command name changes

trashlist-trashrestore-trashempty-trash

trash-puttrash-listtrash-restoretrash-empty


Risultato

Debian --> vecchi nomiUpstream --> nomi nuoviFedora --> Niente!!! (perdita di interesse dal package maintainer)


Lessons Learned

Non cambiare mai il nome dei comandiNon cedere alle pressioni dei revisori (tanto cambiano)

More at: https://github.com/andreafrancia/trash-cli/blob/master/docs/about-the-generic-naming-issue.txt


https://github.com/andreafrancia/trash-cli/blob/master/docs/about-the-generic-naming-issue.txt




Using Bash for functional testing


shunit


Results

Very slow testsUnreadable failure messagesComplex fixture set upTwo test suites (bash+python)


Lessons LearnedTest di Integrazione? Ok, servono ma meglio che siano pochi.Test Funzionali? Meglio che siano omogenei con i test di unitàPer la bash non sono necessarie librerie/framework di test


Bash for Testing(1)

#!/bin/bashset -‐o errexitset -‐o nounset


Bash for Testing(2)

touch footrash-‐put foo

# foo should have been deletedtest ! -‐f foo


Bash for Testing(3)

diff <(trash-‐list) -‐ << EXPECTED_RESULT2012-‐03-‐02 19:43:44 /Users/andrea/foo2012-‐03-‐02 19:44:04 /Users/andrea/fooEXPECTED_RESULT


License


GPL v2+


Lessons Learned


Lessons Learned (1)

Licenze già fatte sono comode ma vanno capite


Lessons Learned (2)

Il papiro all’inizio del file é una rottura


#!/usr/bin/python# list-trash: list trashed files## Copyright (C) 2007,2008 Andrea Francia Trivolzio(PV) Italy## This program is free software; you can redistribute it and/or# modify it under the terms of the GNU General Public License# as published by the Free Software Foundation; either version 2# of the License, or (at your option) any later version.## This program is distributed in the hope that it will be useful,# but WITHOUT ANY WARRANTY; without even the implied warranty of# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the# GNU General Public License for more details.## You should have received a copy of the GNU General Public License# along with this program; if not, write to the Free Software# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA # 02110-1301, USA.

from libtrash import TrashDirectory

for trashed_file in TrashDirectory.trashed_files() : print "%s %s" % (trashed_file.deletion_date, trashed_file.path)

Papiro della GPL


Lessons learned (3)

A volte conviene usare licenze più semplici


DO WHAT THE FUCK YOU WANT TO PUBLIC LICENSE Version 2, December 2004

Copyright (C) 2004 Sam Hocevar <[email protected]>

Everyone is permitted to copy and distribute verbatim or modified copies of this license document, and changing it is allowed as long as the name is changed.

DO WHAT THE FUCK YOU WANT TO PUBLIC LICENSE TERMS AND CONDITIONS FOR COPYING, DISTRIBUTION AND MODIFICATION

0. You just DO WHAT THE FUCK YOU WANT TO.

http://sam.zoy.org/wtfpl/


mailto:[email protected]




Code Hosting Services


sourceforge.net

✉


Lessons LearnedAutomatable release supportFeedback barrierSi può sempre cambiare

sf.netbitbucket

google code GitHub ?


Internals: TrashDir


KDE Gnome

cli XFCE

TrashDir


How it works

~/.local/share/Trash/ +-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐+ info/ ,-‐-‐-‐-‐-‐| [Trash Info] | pippo.txt.trashinfo | Path=/tmp/pippo.txt | files/ | DeletionDate=2012-‐02-‐29T00:13:54| pippo.txt +-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐-‐+ `. `-‐ original file moved here

$ trash-‐put pippo.txt


How it works

~/.local/share/Trash/ info/ pippo.txt.trashinfo pippo_1.txt.trashinfo pippo_2.txt.trashinfo pippo_3.txt.trashinfo files/ pippo.txt pippo_1.txt pippo_2.txt pippo_3.txt

$ touch pippo.txt; trash-‐put pippo.txt # again$ touch pippo.txt; trash-‐put pippo.txt # again$ touch pippo.txt; trash-‐put pippo.txt # again


KDE Gnome

cliXFCE

TrashDir


pippo.txtpippo.txt


open(3)#include <sys/stat.h> #include <fcntl.h> int open(const char *path, int oflag, ... ); ^ O_EXCL | O_CREAT -‐’


os.openimport os

fd = os.open(dest, os.O_RDWR | os.O_CREAT | os.O_EXCL, 0600) os.write(fd, content)os.close(fd)logger.trashinfo_created_as(dest)

return (dest, trash_id)


Conclusioni


Lezioni miste (1)

Utenti: va bene ascoltarli ma la vision la devi avere tu (WONTFIX, parallelo)Promozione per distro: Non vale la pena se poi non lo aggiornano


Lezioni miste (2)

Code Review? Utilissime, basta chiedere in lista.Occhio alle librerie e ai framework (OptionParser, Unipath)


Lezioni miste (3)

Tempo a disposizione? Peggio che al lavoro!

Alta coesione e basso accoppiamento e test veloci sono indispensabili


Lezioni miste (4)È vero che Git sembra più complicato di Subversion?Però

è più facile integrare le patch degli altri meno conflitti inesistenticommit localinon si incasina se sposti una directorycapisce da solo se hai rinominato un file


Lezioni miste (5)

Collaboration tools (wiki, blog , mailing-list)➡Cominciano ad essere usati solo dopo

quando c’e’ veramente tanto interesse➡Si può aspettare a farli.


Branches


Lezione più grande


Come lo farei ora?Beer/Pizza LicenseGitGitHubOutside In TDDNo Wiki, solo .txt tra i sorgentiNessun push promozionale


GrazieAndrea Francia

http://[email protected]

Libero da settembre 2012


http://andreafrancia.it

http://andreafrancia.it



Altre cose


Altro

Singleton LoggerTroppa docs sulla wiki


Tutti i miei sbagli, versione 7 Marzo 2012 al XPUG mi

Technology

Transcript of Tutti i miei sbagli, versione 7 Marzo 2012 al XPUG mi