Red Hat Bugzilla – Bug 462458
Review Request: drupal-workspace - Allows users to have a central place to view and manage their content
Last modified: 2009-11-16 02:37:21 EST
Each user has his/her own workspace, which is accessible by following a
"My workspace" link in the main site navigation, just like the
"My account" link. The workspace interface is tabbed (in the Drupal
6.x version) and shows Content, Comments, and Attachments (attachments
are files the user has uploaded with Drupal core's upload module).
Wow, this is an old one. I know nothing of drupal and there are no specific packaging guidelines to go by, but I suppose I could look at existing approved packages for guidance. However:
The currently recommended version seems to be 1.4 rc1 or somesuch. (Not unexpected since this ticket is 10.5 months old.) Should it be updated before review?
Where do you see that the license is restricted to GPLv2 only? I just see a copy of the GPL with no statement of the version in use, which would imply GPL+. Is there a more explicit indication of the license that I'm missing?
No, there isn't, I flubbed that. :)
Here's the new version:
I'm back to looking over ancient review tickets again. Again, I know zilch about dupal, but nobody else has looked at this in 14 months and fortunately the package still seems to be current, so....
Note that you no longer need BuildRoot: or the first line of %install in Fedora. You can probably remove them unless someone plans to update to drupal 6 in EPEL.
The license in the spec is still GPLv2, though it seems as though you intended
to change it.
* source files match upstream. sha256sum:
* package meets naming and versioning guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* summary is OK.
* description is OK.
* dist tag is present.
* build root is OK.
X license field does not match the actual license.
* license is open source-compatible.
* license text included in package.
* latest version is being packaged.
* BuildRequires are proper (none).
* %clean is present.
* package builds in mock (rawhide, x86_64).
* package installs properly.
* rpmlint is silent.
* final provides and requires are sane:
drupal-workspace = 6.x.1.4-0.rc1.fc12
drupal >= 6.0
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* no generically named files
* code, not content.
* documentation is small, so no -doc subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.
Looks like you're right. . .
I'm not sure what you changed. The license field is still not correct (should be GPL+) , and the other bits like buildroot and the deletion of the buildroot in %install are still there (though I don't know if you intend to remove those).
I changed the license tag to GPLv2+, since Version 2 is what's included and they say that or any later version.
I don't want to remove the buildroot bit, in case someone wants this for EPEL.
Where do they say "that or any later version"? That's what I asked in my first comment; you didn't answer. I grepped for any string I could think of, but I could not find it. I only see the standard GPL text with no statement of the license versioning in the code, which implies GPL+. (I'm sure you've done enough reviews to know that the version of the GPL text included has no bearing on the version of the GPL the code is under.)
Also, I hope you know that EPEL doesn't have drupal 6, and thus this package can't be built for EPEL.
Ok, I obviously got confused, it's not there. Maybe it was in the original or something. In any case, if Version 2 of the GPL is there, and there's no 'or later' anywhere, then it really ought to by GPLv2, not GPL+. Agreed?
Definitely not agreed. Please read the GPL text that's in the package, the last sentence of section 9 in particular. If the source code does not indicate a GPL version, you may use any version ever published. GPL+ is the proper license tag.
Huh. I was not aware of that clause. Learn something new every day. Thanks for pointing that out.
For reference, that is mentioned explicitly in the entry for GPL+ at http://fedoraproject.org/wiki/Licensing
New Package CVS Request
Package Name: drupal-workspace
Short Description: Allows users to have a central place to view and manage their content
Branches: F-12 F-11 F-10
drupal-workspace-6.x.1.4-2.rc1.fc11 has been submitted as an update for Fedora 11.
drupal-workspace-6.x.1.4-2.rc1.fc10 has been submitted as an update for Fedora 10.
drupal-workspace-6.x.1.4-2.rc1.fc12 has been submitted as an update for Fedora 12.
drupal-workspace-6.x.1.4-2.rc1.fc12 has been pushed to the Fedora 12 stable repository. If problems still persist, please make note of it in this bug report.
drupal-workspace-6.x.1.4-2.rc1.fc11 has been pushed to the Fedora 11 stable repository. If problems still persist, please make note of it in this bug report.
drupal-workspace-6.x.1.4-2.rc1.fc10 has been pushed to the Fedora 10 stable repository. If problems still persist, please make note of it in this bug report.