[Bug 631558] Review Request: arduino - An IDE for Arduino-compatible electronics prototyping platforms

2010-10-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=631558

--- Comment #24 from Kevin Fenzi  2010-10-16 17:15:40 EDT ---
Git done (by process-git-requests).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631558] Review Request: arduino - An IDE for Arduino-compatible electronics prototyping platforms

2010-10-15 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=631558

Peter Oliver  changed:

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?

--- Comment #23 from Peter Oliver  2010-10-15 12:11:25 EDT 
---
Package Change Request
==
Package Name: arduino
New Branches: el6
Owners: mavit chitlesh

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631558] Review Request: arduino - An IDE for Arduino-compatible electronics prototyping platforms

2010-10-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=631558

--- Comment #22 from Chitlesh GOORAH  2010-10-12 13:26:48 
EDT ---
Well, we need to get people to use your package. The number of people who will
use Fedora for hardware design is limited. But the number of people who will
use CentOS/RHEL for hardware design is more.

With my experience with FEL, I can tell CentOS/RHEL have 100x more users using
EDA tools than in Fedora. And the userbase of Fedora are geeks while most
certainly you will find hardware engineers runing CentOS/RheL, because its the
OS they use at work.

This is a bit different for software world. 

This is also a reason why I'm migrating FEL to CentOS/RHEL, in order to bring
opensource EDA tools closer to engineers, in hope they might help us move the
wagon along in terms of technology transfer.

I am willing co-maintain this package with you. Do you mind if I request a
branch for EL-6 ?

I invite you to join our FEL mailing list. I'm mostly interested in ASIC
design, but some people like embedded design, a field I don't consider myself
expert. Perhaps you can help us improve this embedded design flow.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631558] Review Request: arduino - An IDE for Arduino-compatible electronics prototyping platforms

2010-10-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=631558

--- Comment #21 from Peter Oliver  2010-10-11 11:21:22 EDT 
---
(In reply to comment #20)
> Peter, I'll email you several times about rebuilding this package for EL-6. 
> Can
> you please tell me at least if you are interested ?

This is my first package, of an application that is yet to reach an API-stable
1.0 release.  I think I'd like to wait a little while to see how things go
before committing to support anything for seven years.

You're welcome to become co-maintainer if you'd like to take this on yourself.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631558] Review Request: arduino - An IDE for Arduino-compatible electronics prototyping platforms

2010-10-10 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=631558

--- Comment #20 from Chitlesh GOORAH  2010-10-10 06:42:48 
EDT ---
Peter, I'll email you several times about rebuilding this package for EL-6. Can
you please tell me at least if you are interested ?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631558] Review Request: arduino - An IDE for Arduino-compatible electronics prototyping platforms

2010-09-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=631558

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|arduino-0019-6.fc14 |arduino-0019-6.fc13

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631558] Review Request: arduino - An IDE for Arduino-compatible electronics prototyping platforms

2010-09-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=631558

--- Comment #19 from Fedora Update System  
2010-09-30 06:28:18 EDT ---
arduino-0019-6.fc13 has been pushed to the Fedora 13 stable repository.  If
problems still persist, please make note of it in this bug report.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631558] Review Request: arduino - An IDE for Arduino-compatible electronics prototyping platforms

2010-09-29 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=631558

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||arduino-0019-6.fc14
 Resolution||ERRATA
Last Closed||2010-09-30 02:12:08

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631558] Review Request: arduino - An IDE for Arduino-compatible electronics prototyping platforms

2010-09-29 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=631558

--- Comment #18 from Fedora Update System  
2010-09-30 02:12:02 EDT ---
arduino-0019-6.fc14 has been pushed to the Fedora 14 stable repository.  If
problems still persist, please make note of it in this bug report.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631558] Review Request: arduino - An IDE for Arduino-compatible electronics prototyping platforms

2010-09-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=631558

q2dg  changed:

   What|Removed |Added

 CC||q...@yahoo.es

--- Comment #17 from q2dg  2010-09-25 06:57:06 EDT ---
Thank you Thank you

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631558] Review Request: arduino - An IDE for Arduino-compatible electronics prototyping platforms

2010-09-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=631558

Chitlesh GOORAH  changed:

   What|Removed |Added

 CC||chitl...@gmail.com

--- Comment #16 from Chitlesh GOORAH  2010-09-25 04:43:22 
EDT ---
Hello Peter,

can you please request branches for EL-6 too ?

thank you,
chitlesh

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631558] Review Request: arduino - An IDE for Arduino-compatible electronics prototyping platforms

2010-09-22 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=631558

Peter Oliver  changed:

   What|Removed |Added

 Blocks||636581

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631558] Review Request: arduino - An IDE for Arduino-compatible electronics prototyping platforms

2010-09-21 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=631558

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #15 from Fedora Update System  
2010-09-21 14:16:54 EDT ---
arduino-0019-6.fc14 has been pushed to the Fedora 14 testing repository.  If
problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update arduino'.  You can provide
feedback for this update here:
https://admin.fedoraproject.org/updates/arduino-0019-6.fc14

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631558] Review Request: arduino - An IDE for Arduino-compatible electronics prototyping platforms

2010-09-21 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=631558

--- Comment #14 from Fedora Update System  
2010-09-21 09:58:13 EDT ---
arduino-0019-6.fc13 has been submitted as an update for Fedora 13.
https://admin.fedoraproject.org/updates/arduino-0019-6.fc13

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631558] Review Request: arduino - An IDE for Arduino-compatible electronics prototyping platforms

2010-09-21 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=631558

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631558] Review Request: arduino - An IDE for Arduino-compatible electronics prototyping platforms

2010-09-21 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=631558

--- Comment #13 from Fedora Update System  
2010-09-21 09:48:56 EDT ---
arduino-0019-6.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/arduino-0019-6.fc14

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631558] Review Request: arduino - An IDE for Arduino-compatible electronics prototyping platforms

2010-09-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=631558

--- Comment #12 from Kevin Fenzi  2010-09-20 23:43:23 EDT ---
Git done (by process-git-requests).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631558] Review Request: arduino - An IDE for Arduino-compatible electronics prototyping platforms

2010-09-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=631558

Peter Oliver  changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #11 from Peter Oliver  2010-09-20 
12:49:23 EDT ---
New Package SCM Request
===
Package Name: arduino
Short Description: An IDE for Arduino-compatible electronics prototyping
platforms
Owners: mavit
Branches: f13 f14
InitialCC:

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631558] Review Request: arduino - An IDE for Arduino-compatible electronics prototyping platforms

2010-09-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=631558

Alexander Kurtakov  changed:

   What|Removed |Added

 Blocks|177841(FE-NEEDSPONSOR)  |

--- Comment #10 from Alexander Kurtakov  2010-09-20 
11:23:35 EDT ---
Dropping FE-NEEDSPONSOR. I've sponsored this contributor.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631558] Review Request: arduino - An IDE for Arduino-compatible electronics prototyping platforms

2010-09-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=631558

Alexander Kurtakov  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #9 from Alexander Kurtakov  2010-09-20 
10:44:05 EDT ---
Thanks, 
This package is APPROVED.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631558] Review Request: arduino - An IDE for Arduino-compatible electronics prototyping platforms

2010-09-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=631558

--- Comment #8 from Peter Oliver  2010-09-20 
10:32:17 EDT ---
Okay, I've named the doc files explicitly.  This means the reference
documentation can no longer be in the same directory (since it gets deleted by
%doc), and some users might not want the documentation anyway, so I've moved it
into a separate subpackage.

Updated files at http://www.triv.org.uk/~mavit/rpm/specs/arduino.spec,
http://www.triv.org.uk/~mavit/rpm/fedora-13/SRPMS/arduino-0019-6.fc13.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631558] Review Request: arduino - An IDE for Arduino-compatible electronics prototyping platforms

2010-09-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=631558

--- Comment #7 from Alexander Kurtakov  2010-09-20 
08:41:08 EDT ---
I would prefer to see license listed directly so it didn't get missed by some
future changes.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631558] Review Request: arduino - An IDE for Arduino-compatible electronics prototyping platforms

2010-09-17 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=631558

--- Comment #6 from Peter Oliver  2010-09-17 
15:48:18 EDT ---
Thanks again.

The package arduino-core contains license.txt and readme.txt, and both are
listed as documentation if you run rpm -qd.  Is this sufficient?

At present, directory /usr/share/doc/arduino-0019/ is being used for
documentation.  If you use %doc, you end up with files in
/usr/share/doc/arduino-core-0019/, which seems messy.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631558] Review Request: arduino - An IDE for Arduino-compatible electronics prototyping platforms

2010-09-17 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=631558

--- Comment #5 from Alexander Kurtakov  2010-09-17 
07:45:27 EDT ---
Review:
OK: rpmlint must be run on every package. There are a number of 
devel-file-in-non-devel-package warnings but as this is an IDE for cross
compiling and these headers/sources are used only for crosscompiling there is
no reason to put them in devel package.
OK: The package must be named according to the Package Naming Guidelines .
OK: The spec file name must match the base package %{name}, in the format
%{name}.spec unless your package has an exemption. 
OK: The package must meet the Packaging Guidelines .
OK: The package must be licensed with a Fedora approved license and meet the
Licensing Guidelines .
OK: The License field in the package spec file must match the actual license.

FIXIT: If (and only if) the source package includes the text of the license(s)
in its own file, then that file, containing the text of the license(s) for the
package must be included in %doc.
There is a lisense.txt file which you should put as %doc.

OK: The spec file must be written in American English. 
OK: The spec file for the package MUST be legible.
OK: The sources used to build the package must match the upstream source, as
provided in the spec URL. Instructions to generate the tarball included.
OK: The package MUST successfully compile and build into binary rpms on at
least one primary architecture.
OK: All build dependencies must be listed in BuildRequires
OK: Packages must NOT bundle copies of system libraries.
OK: A package must own all directories that it creates. 
OK: A Fedora package must not list a file more than once in the spec file's
%files listings.
OK: Permissions on files must be set properly. Executables should be set with
executable permissions, for example. Every %files section must include a
%defattr(...) line. 
OK: Each package must consistently use macros. 
OK: The package must contain code, or permissable content. 
OK: If a package includes something as %doc, it must not affect the runtime of
the application. 
OK: Packages containing GUI applications must include a %{name}.desktop file,
and that file must be properly installed with desktop-file-install in the
%install section. 
OK: Packages must not own files or directories already owned by other packages. 
OK: All filenames in rpm packages must be valid UTF-8. 

So there is just one small issue the missing %doc for lisence.txt it would be
good if you add readme.txt too.
When these are fixed I'll approve the package and sponsor you.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631558] Review Request: arduino - An IDE for Arduino-compatible electronics prototyping platforms

2010-09-17 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=631558

Alexander Kurtakov  changed:

   What|Removed |Added

   Flag||fedora-review?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631558] Review Request: arduino - An IDE for Arduino-compatible electronics prototyping platforms

2010-09-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=631558

--- Comment #4 from Peter Oliver  2010-09-12 
11:50:20 EDT ---
Sorry, I forgot to re-run mock after the previous changes.  The BuildRequires
should all be present, now.  I've also removed the file-based Requires.

Updated SRPM at
http://www.triv.org.uk/~mavit/rpm/fedora-13/SRPMS/arduino-0019-5.fc13.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631558] Review Request: arduino - An IDE for Arduino-compatible electronics prototyping platforms

2010-09-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=631558

--- Comment #3 from Alexander Kurtakov  2010-09-09 
06:04:31 EDT ---
Hi Peter,
You took the easiest approach to use ln -s /usr/share/java app/lib
 but this is crying for problems. You can easily miss a build requirement in
your spec file. The preferred way is to keep your lib folder and create
symlinks for each jar.
It is obviously missing some BuildRequires because it is failing to build on my
machine with the following error:
[mkdir] Created dir:
/home/akurtakov/work/runtime-EclipseApplication/arduino/BUILD/arduino-0019/app/bin
[javac] Compiling 57 source files to
/home/akurtakov/work/runtime-EclipseApplication/arduino/BUILD/arduino-0019/app/bin
[javac]
/home/akurtakov/work/runtime-EclipseApplication/arduino/BUILD/arduino-0019/app/src/processing/app/Editor.java:44:
package gnu.io does not exist
[javac] import gnu.io.*;



Also you're using 
Requires: avr-gcc avr-gcc-c++ avr-libc avrdude /usr/bin/avrdude
/etc/avrdude/avrdude.conf
File based requires like /usr/bin/avrdude should be avoided as much as possible
because they make yum download the filelist db of every package in the
repository which is quite big. Just requiring the packages that contains this
file if preferred.

I'll do the official review once such issues are fixed.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631558] Review Request: arduino - An IDE for Arduino-compatible electronics prototyping platforms

2010-09-08 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=631558

--- Comment #2 from Peter Oliver  2010-09-08 
11:48:12 EDT ---
Thanks.

I have updated the package to make sure that these files are removed from the
source bundle, and re-make them again afterwards.

Spec URL: http://www.triv.org.uk/~mavit/rpm/specs/arduino.spec
SRPM URL:
http://www.triv.org.uk/~mavit/rpm/fedora-13/SRPMS/arduino-0019-4.fc13.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631558] Review Request: arduino - An IDE for Arduino-compatible electronics prototyping platforms

2010-09-07 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=631558

Alexander Kurtakov  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||akurt...@redhat.com
 AssignedTo|nob...@fedoraproject.org|akurt...@redhat.com

--- Comment #1 from Alexander Kurtakov  2010-09-08 
02:30:16 EDT ---
Hi Peter, 
I'll do the review and sponsor you.

First problem - Package in fedora should never use jars/classes build outside
Fedora. And your contain jar files in the app/lib folder. To be sure that
you're clean you have to add
find -name '*.class' -exec rm -f '{}' \;
find -name '*.jar' -exec rm -f '{}' \;

after you %setup line. When this jars are removed the build fails with 
BUILD FAILED
/home/akurtakov/work/runtime-EclipseApplication/arduino/BUILD/arduino-0019/build/build.xml:41:
The following error occurred while executing this line:
/home/akurtakov/work/runtime-EclipseApplication/arduino/BUILD/arduino-0019/build/build.xml:61:
The following error occurred while executing this line:
/home/akurtakov/work/runtime-EclipseApplication/arduino/BUILD/arduino-0019/core/build.xml:12:
taskdef class PAppletMethods cannot be found
 using the classloader AntClassLoader[]

You should also check for any prebuild libraries, remove them and verify they
are build from source.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631558] Review Request: arduino - An IDE for Arduino-compatible electronics prototyping platforms

2010-09-07 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=631558

Peter Oliver  changed:

   What|Removed |Added

 Blocks||177841(FE-NEEDSPONSOR)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review