Re: adoção de pacote
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On 02/18/2007 03:46 PM, Herbert P Fortes Neto wrote:
> Bem, fiz o primeiro pacote. Com poucas
> modificações:
>
> - troca de funções
> - opção de desabilitar a verificação das tabelas
> (só vai saber quem ler o changelog)
Hmmm... eu tinha impressão que essas mudanças eram intrusivas
demais para o Etch, mas fui confirmar no #debian-release na OFTC, o
vorlon (que é o Steve Langasek, o Release Manager) confirmou minhas
suspeitas.
- ----- Begin of IRC Transcript
<faw> Somebody of the release team could check a diff and tell me
if it is suitable for an unblock request? We had a request
to sponsor a Brazilian Developer that would like to adopt
myphpmoney, but I have the impression that his changes are a
little bit way to invasive
<faw> http://www.txtbox.xpg.com.br/myphpmoney/myphpmoney_1.3RC3+dfsg-2.1.diff.gz
<vorlon> faw: er, yes, no way that would be appropriate for etch
<faw> vorlon, thanks ;)
- ----- End of IRC Transcript
> Não troquei o campo 'Mantenedor', o lintian retorna
> sobre o pacote ser orfão, ser de responsabilidade do
> QA. As dependências continuam as mesmas. Para facilitar
> a inclusão no etch. A justificativa para o upload
> seria atualização de funções sem alterar o funcionamento.
Se você _realmente_ pretende manter o myphpmoney, tanto
pro Etch quanto pro Lenny e futuras versões, agora é a hora pra
mudar o campo e adotar o pacote, esta mudança seria aceita pro
Etch.
> Ainda poderia incluir o pt_BR.po que acabei de enviar
> para l10n-portuguese e o brazilian.inc.php que também está
> lá.
Localizações são OK pro etch.
> Acabei de ver que terei que refazer o pacote pelo menos
> mais uma vez. A maioria das palavras 'OUT' tem aspas simples
> mais ainda faltam:
>
> account.php file at lines 562 565 571 573
> http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=394609
Adicionar as aspas corrigiriam o bug e _provavelmente_
seriam aceitas para o etch.
> O end para os arquivos:
[...]
> http://www.txtbox.xpg.com.br/myphpmoney/myphpmoney_1.3RC3+dfsg-2.1.diff.gz
[...]
Ok, vamos às considerações mais importantes sobre o diff.gz,
lembre-se que *todos* os meus comentários são feitos imaginando que
você quer colocar este pacote no etch.
- --- myphpmoney-1.3RC3+dfsg.orig/debian/changelog
+++ myphpmoney-1.3RC3+dfsg/debian/changelog
@@ -1,3 +1,12 @@
+myphpmoney (1.3RC3+dfsg-2.1) unstable; urgency=low
+
+ * Non-maintainer upload.
+ * replace ereg* -> preg*,strpos
+ * took off count(),sizeof() from for,while loops
+ * enable/disable CHK_TABLE in vars.inc.php file
+
+ -- Herbert Parentes Fortes Neto (hpfn) <h_p_f_n@yahoo.com.br> Sat, 18 Feb 2007 09:47:20 -0200
+
Embora o changelog esteja correto, você está mudando coisas
demais, isto é, se você quiser corrigir este pacote pro etch.
- --- myphpmoney-1.3RC3+dfsg.orig/debian/README
+++ myphpmoney-1.3RC3+dfsg/debian/README
@@ -0,0 +1,4 @@
+
+You can disable __CHK_TABLE__ in vars.inc.php file
+to increase velocity.
+
Você vai instalar isso como README.Debian, correto?
- --- myphpmoney-1.3RC3+dfsg.orig/debian/compat
+++ myphpmoney-1.3RC3+dfsg/debian/compat
@@ -1 +1 @@
- -4
+5
Não é hora de mudar compatibilidade, deixe pro Lenny.
- --- myphpmoney-1.3RC3+dfsg.orig/debian/rules
+++ myphpmoney-1.3RC3+dfsg/debian/rules
@@ -5,6 +5,11 @@
+## cp from wmaker/rules
+LIST_PATCHES := find debian/patches -mindepth 1 -maxdepth 1 -name \*.diff -type f | sort
+RLIST_PATCHES := find debian/patches -mindepth 1 -maxdepth 1 -name \*.diff -type f | sort -r
+
+
INSTALL_FILE = install -p -o root -g root -m 644
configure: configure-stamp
@@ -16,14 +21,57 @@
build: build-stamp
- -build-stamp: configure-stamp
+build-stamp: patch-myphpmoney-stamp configure-stamp
dh_testdir
# Nothing to do
touch build-stamp
+## cp from wmaker/rules
+patch-myphpmoney-stamp:
+ @$(LIST_PATCHES) | \
+ while read patch ; do \
+ echo $$patch ; \
+ if head -1 $$patch | grep -q ^Index: ; then \
+ patch -p0 < $$patch ; \
+ else \
+ patch -p1 < $$patch ; \
+ fi ; \
+ echo ; \
+ done
+ touch $@
+
+test-patch-myphpmoney:
+ @$(LIST_PATCHES) | \
+ while read patch ; do \
+ echo $$patch ; \
+ if head -1 $$patch | grep -q ^Index: ; then \
+ patch --dry-run -p0 < $$patch ; \
+ else \
+ patch --dry-run -p1 < $$patch ; \
+ fi ; \
+ echo ; \
+ done
+
+unpatch-myphpmoney:
+ @if [ -e patch-myphpmoney-stamp ] ; then \
+ $(RLIST_PATCHES) | \
+ while read patch ; do \
+ echo $$patch ; \
+ if head -1 $$patch | grep -q ^Index: ; then \
+ patch -p0 -R < $$patch ; \
+ else \
+ patch -p1 -R < $$patch ; \
+ fi ; \
+ echo ; \
+ done ; \
+ rm patch-myphpmoney-stamp ; \
+ fi
+
+
clean:
dh_testdir
dh_testroot
+ debian/rules unpatch-myphpmoney
rm -f build-stamp configure-stamp
# Add here commands to clean up after the build process.
@@ -58,6 +106,9 @@
# Install overrides file
$(INSTALL_FILE) $(CURDIR)/debian/overrides $(CURDIR)/debian/myphpmoney/usr/share/lintian/overrides/myphpmoney
+ # Install contrib files (sqlite)
+ #cp $(CURDIR)/debian/contrib/* $(CURDIR)/debian/myphpmoney/usr/share/doc/myphpmoney/contrib/
+
Qual o objetivo de adicionar linhas comentadas?
# Build architecture-dependent files here.
binary-arch: build install
# We have nothing to do by default.
@@ -92,4 +143,4 @@
dh_builddeb
binary: binary-indep binary-arch
- -.PHONY: build clean binary-indep binary-arch binary install configure
+.PHONY: build clean binary-indep binary-arch binary install configure unpatch-myphpmoney
- --- myphpmoney-1.3RC3+dfsg.orig/debian/control
+++ myphpmoney-1.3RC3+dfsg/debian/control
- -Build-Depends: debhelper (>> 4.1.16)
+Build-Depends: debhelper (>= 5)
De novo, não é hora de mudar o debhelper.
- --- myphpmoney-1.3RC3+dfsg.orig/debian/patches/listop.php.diff
+++ myphpmoney-1.3RC3+dfsg/debian/patches/listop.php.diff
Daqui pra frente, o diff.gz desanda. ;)
São muitas mudanças, incluindo mudanças de patches. Pro
etch não é mais hora de mudar código e fazer melhorias. Corrigir
erros _realmente_ importantes e atualizar traduções são itens que
tem grandes chances de ser aprovados, mudanças além disso correm
o risco de serem bloqueadas, e aí perdemos tempo tanto preparando
o pacote quanto fazendo o upload.
Agora, se sua idéia é cuidar disso pro Lenny, aí tudo bem,
vamos esperar o etch ser lançado e fazemos o upload com suas
mudanças e completando o processo de adoção. Além disso, se você
realmente quer adotar o pacote, mande um ITA.
> Herbert Parentes Fortes Neto (hpfn)
> Linux user number 416100
> 0x9834F79E -- http://pgp.mit.edu/
Abraço,
- --
Felipe Augusto van de Wiel (faw)
"Debian. Freedom to code. Code to freedom!"
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
iD8DBQFF2POHCjAO0JDlykYRAlUVAKDEBJjQ2Ysbrju8dnca1a16ibH43gCgh0+o
InKXVUB+fOBxSPw8KnW9KOQ=
=GSFN
-----END PGP SIGNATURE-----
Reply to: