Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 177049 - dev-java/cos had quotes in DESCRIPTION and java eclasses didn't warn about them
Summary: dev-java/cos had quotes in DESCRIPTION and java eclasses didn't warn about them
Status: RESOLVED WONTFIX
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Java (show other bugs)
Hardware: All Linux
: High normal
Assignee: Java team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-05-04 16:13 UTC by Petteri Räty (RETIRED)
Modified: 2015-01-22 22:59 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Petteri Räty (RETIRED) gentoo-dev 2007-05-04 16:13:20 UTC
betelgeuse@pena ~ $ source /usr/share/cos/package.env
bash: have class library for servlet developers.: command not found

DESCRIPTION=" The com.oreilly.servlet package is the "must have" class library for servlet developers."

Should check and die on these cases or handle them in the eclass by using for example single quotes.
Comment 1 Petteri Räty (RETIRED) gentoo-dev 2007-05-04 16:15:00 UTC
For now I just removed the quotes from DESCRIPTION
Comment 2 Josh Nichols (RETIRED) gentoo-dev 2007-05-19 02:25:15 UTC
This kind of has implications beyond just the java eclasses/tools, because you probably shouldn't be able to have unescaped quotes in general. Of course, our tools should still handle this case.
Comment 3 Patrice Clement (RETIRED) gentoo-dev 2015-01-22 22:59:44 UTC
patrice@freshmint ~/gentoo-x86/dev-java/cos $ grep -A 1 DESCRIPTION cos-20021105-r3.ebuild 
DESCRIPTION=" The com.oreilly.servlet package is the "must have" class library
for servlet developers."
patrice@freshmint ~/gentoo-x86/dev-java/cos $ repoman scan                                                                        

RepoMan scours the neighborhood...
  DESCRIPTION.missing           1
   dev-java/cos/cos-20021105-r3.ebuild
  ebuild.badheader              1

Note: use --include-dev (-d) to check dependencies for 'dev' profiles

Please fix these important QA issues first.
RepoMan sez: "Make your QA payment on time and you'll never see the likes of me."

These kinds of QA errors are now handled by repoman. There's no need to add an extra layer of checks. Closing this bug.