pacman 3.3 API changes

View: New views
8 Messages — Rating Filter:   Alert me  

pacman 3.3 API changes

by Xavier Chantry-2 :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

we had a section in the README about API CHANGES BETWEEN 3.1 AND 3.2
do we need the same for 3.2 -> 3.3?
As you can see in the attached diff, there were quite a few changes as well :P


_______________________________________________
pacman-dev mailing list
pacman-dev@...
http://www.archlinux.org/mailman/listinfo/pacman-dev

API_CHANGES.diff (9K) Download Attachment

Re: pacman 3.3 API changes

by Nagy Gabor-6 :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

> we had a section in the README about API CHANGES BETWEEN 3.1 AND 3.2
> do we need the same for 3.2 -> 3.3?
> As you can see in the attached diff, there were quite a few changes as well :P

I will send a patch for this when we are in "patch freeze". (I have 4
pending patches, all of them change API.)


_______________________________________________
pacman-dev mailing list
pacman-dev@...
http://www.archlinux.org/mailman/listinfo/pacman-dev

Re: pacman 3.3 API changes

by Xavier Chantry-2 :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

On Sat, Jul 11, 2009 at 2:32 PM, Nagy Gabor<ngaba@...> wrote:
>> we had a section in the README about API CHANGES BETWEEN 3.1 AND 3.2
>> do we need the same for 3.2 -> 3.3?
>> As you can see in the attached diff, there were quite a few changes as well :P
>
> I will send a patch for this when we are in "patch freeze". (I have 4
> pending patches, all of them change API.)
>

Great to hear this :)
I think your 2 patches on working should be fine for 3.3.
What about the two for universal, you consider them ready for inclusion?

I believe Dan wanted to include them after 3.3, but I don't know if
this was a final decision.
_______________________________________________
pacman-dev mailing list
pacman-dev@...
http://www.archlinux.org/mailman/listinfo/pacman-dev

Re: pacman 3.3 API changes

by Nagy Gabor-6 :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

> On Sat, Jul 11, 2009 at 2:32 PM, Nagy Gabor<ngaba@...> wrote:
> >> we had a section in the README about API CHANGES BETWEEN 3.1 AND 3.2
> >> do we need the same for 3.2 -> 3.3?
> >> As you can see in the attached diff, there were quite a few changes as well :P
> >
> > I will send a patch for this when we are in "patch freeze". (I have 4
> > pending patches, all of them change API.)
> >
>
> Great to hear this :)
> I think your 2 patches on working should be fine for 3.3.
> What about the two for universal, you consider them ready for inclusion?
>
> I believe Dan wanted to include them after 3.3, but I don't know if
> this was a final decision.

The universal branch is not ready, but I consider the 2 patches ready
for inclusion (in fact, I have focused on fixing of FS#3492 and FS#5798
with the least API changes). However, I had not much time to test it and
the inclusion of a _complete_ universal branch to ~4.0 would be nicer,
so I am fine with Dan's roadmap. (However, I have no clue what to do
with universal transaction at pacman front-end side...)

I will send my "API changes" patch soon, based on my working branch,
feel free to modify it, if needed.

Bye


_______________________________________________
pacman-dev mailing list
pacman-dev@...
http://www.archlinux.org/mailman/listinfo/pacman-dev

[PATCH] API changes between 3.2 and 3.3

by Nagy Gabor-6 :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

>From e9d1686e5e54bff2aad12820e335c10603b307db Mon Sep 17 00:00:00 2001
From: Nagy Gabor <ngaba@...>
Date: Wed, 15 Jul 2009 17:08:28 +0200
Subject: [PATCH] API changes between 3.2 and 3.3

Signed-off-by: Nagy Gabor <ngaba@...>
---
 README |   50 ++++++++++++++++++++++++++++++++++++++++++++++++++
 1 files changed, 50 insertions(+), 0 deletions(-)

diff --git a/README b/README
index 1dadc92..131d9dd 100644
--- a/README
+++ b/README
@@ -246,3 +246,53 @@ API CHANGES BETWEEN 3.1 AND 3.2
 - flags:
     PM_TRANS_FLAG_ALLEXPLICIT, PM_TRANS_FLAG_UNNEEDED and
     PM_TRANS_FLAG_RECURSEALL
+
+
+API CHANGES BETWEEN 3.2 AND 3.3
+===============================
+
+[REMOVED]
+- pmsyncpkg_t struct (pmpkg_t is used for all types of transaction
targets):
+  - alpm_sync_get_pkg()
+  - alpm_sync_get_removes() (use alpm_pkg_get_removes() instead)
+- HoldPkg handling (it is the front-end's task):
+  - alpm_option_get_holdpkgs()
+  - alpm_option_add_holdpkg()
+  - alpm_option_set_holdpkgs()
+  - alpm_option_remove_holdpkg()
+  - PM_TRANS_CONV_REMOVE_HOLDPKG conversation
+- Print URIs feature (it is the front-end's task):
+  - flag: PM_TRANS_FLAG_PRINTURIS
+  - event: PM_TRANS_EVT_PRINTURI
+- alpm_delta_get_from_md5sum() and alpm_delta_get_to_md5sum()
+- alpm_sync_sysupgrade()
+- error codes:
+    PM_ERR_TRANS_COMMITING, PM_ERR_TRANS_DOWNLOADING, PM_ERR_PKG_LOAD,
+    PM_ERR_PKG_CANT_FRESH, PM_ERR_GRP_NOT_FOUND, PM_ERR_USER_ABORT,
+    PM_ERR_INTERNAL_ERROR, PM_ERR_DB_SYNC, PM_ERR_PKG_HOLD and
+    PM_ERR_LIBDOWNLOAD
+
+[CHANGED]
+- function renames:
+  - alpm_db_getpkgcache() -> alpm_db_get_pkgcache()
+  - alpm_db_getgrpcache() -> alpm_db_get_grpcache()
+  - alpm_dep_get_string() -> alpm_dep_compute_string()
+  - alpm_get_md5sum() -> alpm_compute_md5sum()
+  - alpm_checkdbconflicts() -> alpm_checkconflicts()
+- alpm_trans_sysupgrade() has a new enable_downgrade parameter
+- alpm_checkdeps() and alpm_checkconflicts() require local package list
instead
+  of local database
+- the to-be-upgraded package is passed to the callback function with
+  PM_TRANS_EVT_UPGRADE_START (as the second parameter)
+- the "requiredby" package is never passed to the callback function
with
+  PM_TRANS_CONV_INSTALL_IGNOREPKG (the second parameter is always NULL)
+
+[ADDED]
+- alpm_db_set_pkgreason()
+- alpm_pkg_get_db()
+- alpm_pkg_get_removes()
+- conversation: PM_TRANS_CONV_REMOVE_PKGS (remove unresolvable targets)
+- flag: PM_TRANS_FLAG_NOLOCK (do not lock database)
+- error codes:
+    PM_ERR_SERVER_NONE, PM_ERR_TRANS_NOT_LOCKED, PM_ERR_PKG_IGNORED and
+    PM_ERR_LIBFETCH
--
1.6.0.3



_______________________________________________
pacman-dev mailing list
pacman-dev@...
http://www.archlinux.org/mailman/listinfo/pacman-dev

Re: [PATCH] API changes between 3.2 and 3.3

by Nagy Gabor-6 :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Special thanks to my mail client for formatting this. See my git repo
for the applicable patch.


_______________________________________________
pacman-dev mailing list
pacman-dev@...
http://www.archlinux.org/mailman/listinfo/pacman-dev

Re: [PATCH] API changes between 3.2 and 3.3

by Xavier Chantry-2 :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

On Wed, Jul 15, 2009 at 5:26 PM, Nagy Gabor<ngaba@...> wrote:

> >From e9d1686e5e54bff2aad12820e335c10603b307db Mon Sep 17 00:00:00 2001
> From: Nagy Gabor <ngaba@...>
> Date: Wed, 15 Jul 2009 17:08:28 +0200
> Subject: [PATCH] API changes between 3.2 and 3.3
>
> Signed-off-by: Nagy Gabor <ngaba@...>
> ---
>  README |   50 ++++++++++++++++++++++++++++++++++++++++++++++++++
>  1 files changed, 50 insertions(+), 0 deletions(-)
>
> diff --git a/README b/README
> index 1dadc92..131d9dd 100644
> --- a/README
> +++ b/README
> @@ -246,3 +246,53 @@ API CHANGES BETWEEN 3.1 AND 3.2
>  - flags:
>     PM_TRANS_FLAG_ALLEXPLICIT, PM_TRANS_FLAG_UNNEEDED and
>     PM_TRANS_FLAG_RECURSEALL
> +
> +
> +API CHANGES BETWEEN 3.2 AND 3.3
> +===============================
> +
> +[REMOVED]
> +- pmsyncpkg_t struct (pmpkg_t is used for all types of transaction
> targets):
> +  - alpm_sync_get_pkg()
> +  - alpm_sync_get_removes() (use alpm_pkg_get_removes() instead)
> +- HoldPkg handling (it is the front-end's task):
> +  - alpm_option_get_holdpkgs()
> +  - alpm_option_add_holdpkg()
> +  - alpm_option_set_holdpkgs()
> +  - alpm_option_remove_holdpkg()

you also need to remove the reference to holdpkgs in the same README
file, as you mentioned earlier.
I did not find any other updates needed.
_______________________________________________
pacman-dev mailing list
pacman-dev@...
http://www.archlinux.org/mailman/listinfo/pacman-dev

Re: [PATCH] API changes between 3.2 and 3.3

by Dan McGee :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

On Wed, Jul 22, 2009 at 4:01 AM, Xavier<shiningxc@...> wrote:

> On Wed, Jul 15, 2009 at 5:26 PM, Nagy Gabor<ngaba@...> wrote:
>> >From e9d1686e5e54bff2aad12820e335c10603b307db Mon Sep 17 00:00:00 2001
>> From: Nagy Gabor <ngaba@...>
>> Date: Wed, 15 Jul 2009 17:08:28 +0200
>> Subject: [PATCH] API changes between 3.2 and 3.3
>>
>> Signed-off-by: Nagy Gabor <ngaba@...>
>> ---
>>  README |   50 ++++++++++++++++++++++++++++++++++++++++++++++++++
>>  1 files changed, 50 insertions(+), 0 deletions(-)
>>
>> diff --git a/README b/README
>> index 1dadc92..131d9dd 100644
>> --- a/README
>> +++ b/README
>> @@ -246,3 +246,53 @@ API CHANGES BETWEEN 3.1 AND 3.2
>>  - flags:
>>     PM_TRANS_FLAG_ALLEXPLICIT, PM_TRANS_FLAG_UNNEEDED and
>>     PM_TRANS_FLAG_RECURSEALL
>> +
>> +
>> +API CHANGES BETWEEN 3.2 AND 3.3
>> +===============================
>> +
>> +[REMOVED]
>> +- pmsyncpkg_t struct (pmpkg_t is used for all types of transaction
>> targets):
>> +  - alpm_sync_get_pkg()
>> +  - alpm_sync_get_removes() (use alpm_pkg_get_removes() instead)
>> +- HoldPkg handling (it is the front-end's task):
>> +  - alpm_option_get_holdpkgs()
>> +  - alpm_option_add_holdpkg()
>> +  - alpm_option_set_holdpkgs()
>> +  - alpm_option_remove_holdpkg()
>
> you also need to remove the reference to holdpkgs in the same README
> file, as you mentioned earlier.
> I did not find any other updates needed.

Sounds about right. I fully intend to pull this patch, I just haven't yet.

-Dan
_______________________________________________
pacman-dev mailing list
pacman-dev@...
http://www.archlinux.org/mailman/listinfo/pacman-dev