Package Details: motivewave 6.9.2-1

Git Clone URL: https://aur.archlinux.org/motivewave.git (read-only, click to copy)
Package Base: motivewave
Description: Advanced trading and charting application.
Upstream URL: https://www.motivewave.com
Keywords: charting forex stocks
Licenses: custom
Submitter: sundbp
Maintainer: xavierbaez
Last Packager: xavierbaez
Votes: 4
Popularity: 0.008333
First Submitted: 2020-11-20 22:16 (UTC)
Last Updated: 2024-04-17 03:52 (UTC)

Dependencies (6)

Required by (0)

Sources (1)

Latest Comments

« First ‹ Previous 1 2 3 4 5 6 7 .. 9 Next › Last »

aliu commented on 2023-09-04 22:01 (UTC) (edited on 2023-09-04 22:05 (UTC) by aliu)

@xavierbaez has taken this to aur-general.

muflone commented on 2023-09-04 18:54 (UTC)

@xavierbaez your choice:

1) you delete your last comment

2) you keep the comment and I'll suspend your account for aggressive and childish behavior towards people helping you

muflone commented on 2023-09-04 18:47 (UTC) (edited on 2023-09-04 18:55 (UTC) by muflone)

@xavierbaez no one insulted you, me and xiota helped you in many ways but you continue arguing xiota is sabotaging you and you're still keeping an offensive and ungrateful approach with others that are spending their time in helping you to fix your package.

So please stop immediately this approach with other people or I'll suspend your account or give this package to someone more skilled to fix the package.

All the things xiota told you were correct, this is the whole point. Simply you're doing multiple errors in fixing it and you blame xiaota for your own errors.

In the next hours I'll take this package and I'll fix it as you don't seem to be able to fix it (I can still see some errors) by yourself.

I'd suggest you to add a co-maintainer to this package to help you in fixing it, your own choice, this won't be enforced at the moment.

xavierbaez commented on 2023-09-04 17:08 (UTC)

Maybe you should stop 🛑 this package builds And the makepkg --printsrcinfo >| .SRCINFO Also works It builds The yum install works So what’s you’re problem? You were told to stop insulting me You created a package which is a duplicate And nobody asked you to nit pick on every single line of code like you’re the owner of this package 📦 Yesterday I came back from church to make your changes Today at 3 AM you want more changes And nobody designated you as code reviewer of motivewave package I can explain the main problem You say Problem 1 - makepkg --printsrcinfo >| .SRCINFO Problem 1 fixed Then you complain about Problem 2 Then Problem 3 And insults again and again and again It’s like dealing with a terrorist

xiota commented on 2023-09-04 16:45 (UTC) (edited on 2023-09-04 16:56 (UTC) by xiota)

@muflone

Given xavierbaez's total inability to fix this package, perhaps someone else should be assigned to maintain it. I gave a step-by-step tutorial, along with a link to a working, tested PKGBUILD, and he still could not figure it out. I would expect any competent maintainer to have had this done within minutes, not spend over a week arguing about regenerating .SRCINFO, posting potentially dangerous code, and wasting time on other irrelevant minutiae.

@xavierbaez

I have already told you so many times during multiple days this does NOT work

source=("${pkgname}-${pkgver}.deb::https://downloads.motivewave.com/builds/${build_id}/motivewave${pkgver}_amd64.deb")

Right. Because in every case, including the above, you have included numerous typos.

I’m waiting for your package to get deleted

Don't worry about the other package. A TU has already said it will be deleted. He is probably waiting for this package to be fixed before doing so. Arguing with him and refusing to fix this package is delaying things.

you can send me an email instead of using comments as a chat

This is the appropriate place to post problems with the package that need to be fixed.

I do not intend to flood my inbox with more messages from you than I'm already receiving.

why are you code reviewing obsessively line by line👨🏻‍💻

You have extreme inability to follow instructions that any competent maintainer would have already completed. To rule out misunderstanding, I simplified the instructions as much as possible.

You have been suggesting code that breaks the package man

The resulting PKGBUILD had already been tested.

xavierbaez commented on 2023-09-04 14:30 (UTC)

I have already told you so many times during multiple days this does NOT work

source=("${pkgname}-${pkgver}.deb::https://downloads.motivewave.com/builds/${build_id}/motivewave${pkgver}_amd64.deb")

And the conflicts with motivewave-latest-bin I’m waiting for your package to get deleted, which for some reason you created a duplicate and it’s still not deleted.

And you can send me an email instead of using comments as a chat

And why are you code reviewing obsessively line by line👨🏻‍💻 You have been suggesting code that breaks the package man.

xiota commented on 2023-09-04 13:09 (UTC) (edited on 2023-09-04 13:23 (UTC) by xiota)

@xavierbaez

Start with a fresh checkout: \rm -rf motivewave ; git clone ssh://aur@aur.archlinux.org/motivewave.git ; cd motivewave

Now we will go through the PKGBUILD line by line.

1: Maintainer is spelled incorrectly. Add previous contributor back.

# Maintainer:  <xavierbaez at gmail dot com>
# Contributor: Patrik Sundberg <patrik.sundberg at gmail dot com>

2: okay, empty line

3: okay: pkgname=motivewave

4: delete pkgname_duplicate

5: okay: pkgver=6.7.10

6: increment: pkgrel=7

7: okay: pkgdesc="Advanced trading and charting application."

8: okay: arch=('x86_64')

9: okay: url="https://www.motivewave.com"

10: okay: license=('custom')

11: Leave alone for now. Review and remove unnecessary depends later. depends=(...)

12: okay: _build_id=500

13: delete _wabas

14: change:

source=("${pkgname}-${pkgver}.deb::https://downloads.motivewave.com/builds/${_build_id}/motivewave_${pkgver}_amd64.deb")

15: okay: sha512sums=(...) (Note: Upstream does provide an md5 hash. So there's no need to generate your own hashes every release.)

16: delete provides

17: delete conflicts

18: okay, empty line

19: okay: package() {

20: add option: bsdtar --no-same-owner -xf data.tar.xz -C "$pkgdir/"

21: okay, empty line

22: delete: find/chmod

23: delete: chown

24: okay: mkdir -pv "$pkgdir/usr/bin"

25: okay: ln -sv "/usr/share/motivewave/run.sh" "$pkgdir/usr/bin/motivewave"

26: okay: install -Dm644 -t "$pkgdir/usr/share/licenses/$pkgname" "$pkgdir/usr/share/$pkgname/license.html"

27: okay: }

Now save and test modified PKGBUILD. If it works, run makepkg --printsrcinfo >| .SRCINFO. Then commit changes.

muflone commented on 2023-09-04 07:52 (UTC) (edited on 2023-09-04 07:54 (UTC) by muflone)

What me and xiota told you to fix the source URL uses a variable called build_id, so you have to set this variable and give it the correct value (500 for this release).

Also better to replace build_id with _build_id, as variable name and in the URL.

_build_id=500
source=("${pkgname}-${pkgver}.deb::https://downloads.motivewave.com/builds/${_build_id}/motivewave${pkgver}_amd64.deb")

No one is sabotaging you, don't think xiota wants this package as defective. Simply you're not good at maintaining it and he's giving you help and hints to have the package working properly.

I've unpinned all the messages, because you haven't done after my three requests.

xavierbaez commented on 2023-09-04 05:00 (UTC)

Latest build provided @mufrone the other guy is doing 'code review' on this package and his code reviews are actually messing up the build source=("${pkgname}-${pkgver}.deb::https://downloads.motivewave.com/builds/${build_id}/motivewave${pkgver}_amd64.deb") This does NOT WORK

This happens when you do: makepkg --printsrcinfo > .SRCINFO

I am not sure what to do, but I do have a job and spent 12 hours on a Sunday doing thing to please one guy, that sends suggestions that don't even work.