New BitBucket push-to-deploy excludes over-ridden packages

deployment
1

Hi, I’m just trying out the excellent new push-to-deploy on Galaxy and have come up against an issue.

I have a few Meteor packages that I have had to modify and as a result I git clone them from my own GitHub repository into a packages folder in my main project. The project itself is in BitBucket, but as far as I can tell the fact that I have modules brought in from GitHub, should not really break things (they are all publicly accessible)? Here’s what it looks like in BB:

Screenshot 2023-01-05 at 11.19.29 1
Screenshot 2023-01-05 at 11.19.29 11106×1332 89.3 KB

But I think the build does not draw in my versions of these modules but, instead, the original, unmodified versions. I only infer this as the build log shows errors that are consistent with using the original versions.

I’d really like to see a more detailed trace of the build process to see what it is doing…

Any thoughts?

2

@denyhs is probably the best person, I think, to chime in.

3

Hi @dthwaite. Thank you for reporting this issue.

I was the one that implemented the P2D for Bitbucket, so let me help you with this problem.

Have you tried this repo using Github? Do you have the same issues there?
Also, what is your hostname to help me figure out more about your app?

Thank you
Henrique

4

Hi Henrique,

No, I’ve not tried GitHub yet - a bit of a hassle, tbh, so will only do so if really necessary.
App is mmm-staging.eu.meteorapp.com - perhaps you can look at the last failed build, around 4 hours ago…

Thanks

Dominic

5

Hi @hschmaiske ,

Have you been able to investigate this yet?

Is it necessary for me to deploy to GitHub and try it from there?

Thanks

Dominic

6

To confirm, I still have the issue with a Github build …

7

Hi @dthwaite sorry for the late reply.
Are you using submodules for those packages?

8

Hi @hschmaiske ,

Yes indeed - sorry I should have mentioned that …

9

Oh okay! So I think I know what happened. I’ll work on that this week and let you know as soon as it is fixed

10

Hi @hschmaiske

Any progress on this one?

Thanks

Dominic

1 Like
11

Hi @dthwaite, not yet
I wasn’t able to work on this, but I’ll make sure we prioritize this to do asap

12

Thank you @hschmaiske

Appreciate it. I’m not in a huge rush so mark as medium priority :slight_smile:

Just will be very cool to have PTD up and running

1 Like