Hi - To make a profile you make a .profile file and a drush make file. You don't check in all of Drupal. Please read about how to make a profile.

Because this breaks the testbots (by pretending to be Drupal core) I have unpublished your release.

Please let me know if you have any questions.

-Randy

CommentFileSizeAuthor
#11 uberpaltw-fix-ubercart-6.x-2.7-shipping.patch5.86 KBjamesliu78
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

charlesc’s picture

charlesc CreditAttribution: charlesc commented
Assigned: Unassigned » jamesliu78

Randy,
Thank you for your reminder. We need to make it a profile only right here and release the distribution in our own project site.

jamesliu78’s picture

jamesliu78
Primary language Chinese, Traditional
CreditAttribution: jamesliu78 commented
Version: 6.x-1.0 » 6.x-1.1
Status: Active » Closed (fixed)

Randy,
The issue already fix on 6.x-1.1, thank you for your reminder.

TR’s picture

TR CreditAttribution: TR commented
Priority: Normal » Major
Status: Closed (fixed) » Active

No, it's not fixed in 6.x-1.x - you still have a full copy of Drupal in your profile.

jamesliu78’s picture

jamesliu78
Primary language Chinese, Traditional
CreditAttribution: jamesliu78 commented

Sorry TR,
I'm sure there have no drupal core files in the profile.
http://ftp.drupal.org/files/projects/uberpaltw-6.x-1.1.tar.gz

6.x-1.1 http://drupal.org/node/1393454

The copy of ubercart 6.x-2.6 is need fix some issue for local to use.

Can you give more detailed description of issue?
Thank you very much.

jamesliu78’s picture

jamesliu78
Primary language Chinese, Traditional
CreditAttribution: jamesliu78 commented

I found the "drupal-org.make" file make this happened.
Should I remove it?

rfay’s picture

rfay
Primary language English
Location Palisade, CO, USA
CreditAttribution: rfay commented

The core files have in fact now been removed from 6.x-1.x... But you also should remove the "ubercart" directory and that will just be added using the drush make.

jamesliu78’s picture

jamesliu78
Primary language Chinese, Traditional
CreditAttribution: jamesliu78 commented

Randy,
About the "ubercart" project, we keep it for fix some problem for local use.

We will report this issue to ubercart project later, and find how to use patch in drush make as soon as possible.

rfay’s picture

rfay
Primary language English
Location Palisade, CO, USA
CreditAttribution: rfay commented

@jamesliu78 the way you deal with that is to use (as you say) patches to be applied by drush make. At least that's my understanding. Thanks for working on this!

TR’s picture

TR CreditAttribution: TR commented

While the core files have been removed from the *repository* for this project, as @rfay said in #6, the core files are still present in the *release* which is linked on the project page. The download link on the project page gives you http://ftp.drupal.org/files/projects/uberpaltw-6.x-1.1-core.tar.gz, which contains a complete Drupal release. This is not how other installation profiles works - I checked the top five installation profiles and none of them contains a Drupal release. I've never created an installation profile release myself so I can't tell you how to fix this, but the Drupal files shouldn't be in the tar.gz.

rfay’s picture

rfay
Primary language English
Location Palisade, CO, USA
CreditAttribution: rfay commented

@TR I think that's the way it's supposed to be. The drush make file builds a full drupal release. That's how Commerce Kickstart works, for example. The code contains a drush make file that builds the thing. And then you can download a full distribution. Just checked and the 6.x-1.1 tag for uberpaltw does *not* contain Drupal files.

jamesliu78’s picture

jamesliu78
Primary language Chinese, Traditional
CreditAttribution: jamesliu78 commented
Status: Active » Needs review
FileSize
uberpaltw-fix-ubercart-6.x-2.7-shipping.patch5.86 KB

I finally create a patch for ubercart modules.
It will be fixed in 6.x-1.2.