[New Upstream Release] Electric has released 8.06

Bug #242720 reported by Aanjhan Ranganathan
6
Affects Status Importance Assigned to Milestone
electric (Ubuntu)
Fix Released
Wishlist
Unassigned

Bug Description

Changelog file attached. Debian unstable also has old release.

Revision history for this message
Aanjhan Ranganathan (aanjhan) wrote :
Revision history for this message
Aanjhan Ranganathan (aanjhan) wrote :

The old version is present because Java was not GPL compatible before. Now Sun Java is GPL Compatible and hence I guess the new release can be packaged. Please free to point out if I have missed something.

Revision history for this message
Onkar Shinde (onkarshinde) wrote :

I will try to work on this in next week.

Changed in electric:
assignee: nobody → onkarshinde
Revision history for this message
Onkar Shinde (onkarshinde) wrote :

Please find attached .diff.gz for latest upstream version 8.07.

Changed in electric:
assignee: onkarshinde → nobody
status: New → Confirmed
Revision history for this message
Onkar Shinde (onkarshinde) wrote :

I forgot to add LP: #xxxxxx entry in changelog. Can the uploader please take care of that?

Maia Everett (linneris)
Changed in electric:
assignee: nobody → sikon
importance: Undecided → Wishlist
status: Confirmed → In Progress
Revision history for this message
Maia Everett (linneris) wrote :

From a quick look (haven't actually built it):

- The repacked orig.tar.gz contains binary .class files. Please make get-orig-source remove them.
- Consider using java-wrappers instead of a custom launcher.
- Instead of "TODO" (which should be "TODO.Debian" anyway), why won't you actually add the icon?
- The files *.debhelper are autogenerated, are you sure they are needed?
- Consider using dh_install. This way, you can get rid of debian/dirs. If you're going to keep the custom wrapper, you can avoid the need for renaming by moving it to something like debian/wrappers/electric.
- debian/electric.desktop does not pass desktop-file-validate
- It's pointless to include ChangeLog.txt in docs, since it's installed with dh_installchangelogs (and if it isn't, use DEB_INSTALL_CHANGELOGS_ALL = ChangeLog.txt)

Changed in electric:
assignee: sikon → onkarshinde
status: In Progress → Incomplete
Revision history for this message
Onkar Shinde (onkarshinde) wrote :

Please find attached the latest .diff.gz.

Following are the changes done.

- Added LP: #xxxxxx in debian/changelog.
- Changed build dependency to default-jdk, runtime dependency to default-jre. Changed JAVA_HOME accordingly.
- Added ChangeLog.txt to DEB_INSTALL_CHANGELOGS_ALL. Removed from debian/electric.docs. It was not being installed automatically due to the extension.
- Fixed get-orig-source to delete *.class files. .orig.tar.gz file reduced by almost 7M. :-D
- Got rid of debian/dirs with help of debian/install. Moved custom launcher to debian/wrappers directory. No need of install target in debian/rules file now.
- Fixed debian/electric.desktop file.
- Added file debian/README.source to comply with standards version 3.8.0.
- Changed file names debian/electric_icon.svg to debian/electric.svg and debian/TODO to debian/TODO.Debian.
- *.debhelper files do not look relevant. Hence I removed them.

Following are the changes not done.
- Couldn't figure out how to use java-wrappers. Will do it sometime later.
- Debian menu file requires XPM icon which is not available. Hence didn't do the change specified in TODO.Debian.

Changed in electric:
assignee: onkarshinde → nobody
status: Incomplete → Confirmed
Revision history for this message
Onkar Shinde (onkarshinde) wrote :

Please find attached latest debdiff.

1. Fixed lintian warnings.
2. Added openjdk to build dependencies. Added home directories to rules file. Will allow easy backport to hardy.
3. Did a minor change to get-orig-source in the part that cleans *.class files.

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package electric - 8.07-0ubuntu1

---------------
electric (8.07-0ubuntu1) intrepid; urgency=low

  * New Upstream version. Please check changelog for details. (LP: #242720)
  * debian/control
    - Add build dependencies *-jdk, cdbs and bsh.
    - Remove build dependency dpatch. We will be using CDBS simple patchsys.
    - Refreshed runtime dependencies to default-jre | java2-runtime and bsh.
    - Added home page field.
    - Standard version 3.8.0.
    - Modify Maintainer value to match the DebianMaintainerField
      specification.
    - Changed email address for original maintainer to indicate who has
      refreshed the packaging.
  * debian/rules
    - Revamped to use cdbs.
    - Added get-orig-source target.
  * debian/patches
    - 00list, 02_sensible-browser.dpatch, 01_errors-numbers.dpatch,
      03_manpage.dpatch - Deleted, not relevant anymore.
    - 01_fix_build_xml.patch - Patch to fix the build.xml.
  * debian/ant.properties
    - File to set various compilation properties.
  * debian/electric.1
    - Remove the entry that causes lintian warning.
  * debian/electric.desktop
    - Change as suggested by desktop-file-validate.
  * debian/electric.docs
    - Updated as per changes in file names.
  * debian/electric.svg
    - Name changed from electric_icon.svg.
  * debian/install
    - Added appropriate locations for jar file, desktop file and wrapper shell
      script.
  * debian/README.source
    - Added to comply with standards version 3.8.0.
  * debian/TODO.Debian
    - Name changed form TODO.
  * debain/wrapper/electric
    - Wrapper shell script to launch the application.
  * debian/manpages
    - Added for installation of manpage.
  * debian/watch
    - Updated to match jar files instead of older tar.gz files.
  * debian/dirs
    - Removed, not needed anymore.
  * debian/{electric.doc-base, electric.examples, substvars}
    - Removed, not relevant anymore.
  * debian/*.debhelper
    - Removed auto generated files. Not relevant anymore.

 -- Onkar Shinde <email address hidden> Wed, 23 Jul 2008 02:09:53 +0530

Changed in electric:
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.