Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0014755
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] A. Platformminorsometimes2010-10-04 12:032011-10-07 13:06
ReporterjpabloaeView Statuspublic 
Assigned Togorka_gil 
PrioritynormalResolutionfixedFixed in Versionpi
StatusclosedFix in branchpiFixed in SCM revision
ProjectionnoneETAnoneTarget Version2.50MP23
OSLinux 32 bitDatabasePostgreSQLJava version1.6.0_21
OS Version2.6.34Database version8.4.4Ant version1.8.0
Product VersionpiSCM revision331c9ca70ae9 
Review Assigned To
Web browser
ModulesCore
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0014755: Warnings with ant tasks in Ant 1.8

DescriptionIf you run Openbravo ERP Ant tasks using using Ant version 1.8.x then it shows warning every time a javac task is run, such as this one:

[javac] /opt/OpenbravoERP-2.50/openbravo-erp/src-core/build.xml:47: warning: 'includeantruntime' was not set, defaulting to build.sysclasspath=last; set to false for repeatable builds
Proposed SolutionStarting from Ant 1.8.0 includeantruntime has been set to "true" by default and it shows a warning if it's not explicitly set. Setting it to false means it doesn't include the ant runtime libs when running javac, so it's more consistent accross systems.
TagsNo tags attached.
Attached Filespatch file icon ant18.patch [^] (16,810 bytes) 2010-10-04 12:07 [Show Content]

- Relationships Relation Graph ] Dependency Graph ]

-  Notes
(0031567)
jpabloae (reporter)
2010-10-04 12:07

I've tested this with both the old behavior (includeantruntime="true") and the recommended one (includeantruntime="false") and they work well. Using 1.7.1 and 1.8.0 in both cases.

My take here is to set it to "false", so that we have a more consistent behavior. I don't see risks here, given that it just applies to the "javac" tasks. I attach a patch that handles these warnings.

Stefan: could you please review it and state your opinion on this matter?
(0031568)
jpabloae (reporter)
2010-10-04 12:08

Additional information: this is relevant because these warnings are shown if you use Ant 1.8.0, and this is the version shipped in the Ubuntu Maverick 10.10 distribution.
(0038542)
shuehner (administrator)
2011-06-27 17:07

Hi,

this should be solved for pi already. Question is if something similar should be backported for 2.50 also or not?
(0041544)
dmitry_mezentsev (developer)
2011-10-07 11:05

Hello All,

This issue is in a feedback status for quite a while.
At the same time based on the feedback from Stefan it is fixed.
Could you please verify?
And answering Stefan´s question - we don´t want to backport it.
(0041550)
shuehner (administrator)
2011-10-07 11:46

Hi dmitry,
the issue is solved in pi. If we don't want to backport it we should just reject/close this issue here (as it's only about 2.50).

Note:
- nothing broken if not fixed
- but some warnings shown always on compile with newer ant version which is confusing/annoying to people
(0041556)
dmitry_mezentsev (developer)
2011-10-07 13:06

Issue is fixed in 3.0.

- Issue History
Date Modified Username Field Change
2010-10-04 12:03 jpabloae New Issue
2010-10-04 12:03 jpabloae Assigned To => jpabloae
2010-10-04 12:03 jpabloae Status new => scheduled
2010-10-04 12:03 jpabloae fix_in_branch => pi
2010-10-04 12:03 jpabloae Issue Monitored: shuehner
2010-10-04 12:07 jpabloae Note Added: 0031567
2010-10-04 12:07 jpabloae File Added: ant18.patch
2010-10-04 12:08 jpabloae Note Added: 0031568
2011-06-27 17:07 shuehner Note Added: 0038542
2011-06-27 17:07 shuehner Status scheduled => feedback
2011-08-24 12:57 jpabloae Assigned To jpabloae => gorka_gil
2011-10-07 11:05 dmitry_mezentsev Note Added: 0041544
2011-10-07 11:46 shuehner Note Added: 0041550
2011-10-07 13:06 dmitry_mezentsev Note Added: 0041556
2011-10-07 13:06 dmitry_mezentsev Status feedback => closed
2011-10-07 13:06 dmitry_mezentsev Resolution open => fixed
2011-10-07 13:06 dmitry_mezentsev Fixed in Version => pi


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker