Re: [arch-general] [PATCH] gdisk patches

2010-07-17 Thread Jérôme M. Berger
Mark Pustjens wrote:
> On Sat, 17 Jul 2010, Ionuț Bîru wrote:
> 
>> On 07/17/2010 01:23 AM, Mark Pustjens wrote:
>>>  On Fri, 16 Jul 2010, Ionuț Bîru wrote:
>>>
>>> >  On 07/16/2010 10:11 PM, Mark Pustjens wrote:
>>> > >  Hi List,
>>> > > > >  Below a patch wich fixes two bugs in the gdisk package. They
>>> have been
>>> > >  submitted uptream but not yet been accepted.
>>> > >  then we don't need them if upstream did that
>>>
>>>  They didn't do anything :) Submitted only a few hours ago.
>>>
>>>
>>
>> post a link to the bug report or mailing list so we can follow the
>> progress
> 
> As i said in my original post: there is no place for a bug report, nor
> is there a mailing list.
> 
http://sourceforge.net/sendmessage.php?touser=238224

But you're right, it's not easy to find and I can't guarantee that
the dev will get the message or that he'll answer...

Jerome
-- 
mailto:jeber...@free.fr
http://jeberger.free.fr
Jabber: jeber...@jabber.fr



signature.asc
Description: OpenPGP digital signature


Re: [arch-general] [PATCH] gdisk patches

2010-07-17 Thread Mark Pustjens

On Sat, 17 Jul 2010, Ionuț Bîru wrote:


On 07/17/2010 01:23 AM, Mark Pustjens wrote:

 On Fri, 16 Jul 2010, Ionuț Bîru wrote:

>  On 07/16/2010 10:11 PM, Mark Pustjens wrote:
> >  Hi List,
> > 
> >  Below a patch wich fixes two bugs in the gdisk package. They have been

> >  submitted uptream but not yet been accepted.
> 
>  then we don't need them if upstream did that


 They didn't do anything :) Submitted only a few hours ago.




post a link to the bug report or mailing list so we can follow the progress


As i said in my original post: there is no place for a bug report, nor is 
there a mailing list.




--
Ionuț



Greetings/Groetjes

Mark Pustjens

--
'It's a lovely morning, lads,' he said. 'I feel like a million dollars. Don't
you?'
There was a murmur of reluctant agreement.
'Good,' said Cohen. 'Let's go and get some.'
  (Interesting Times)

Re: [arch-general] [PATCH] gdisk patches

2010-07-16 Thread Ionuț Bîru

On 07/17/2010 01:23 AM, Mark Pustjens wrote:

On Fri, 16 Jul 2010, Ionuț Bîru wrote:


On 07/16/2010 10:11 PM, Mark Pustjens wrote:

Hi List,

Below a patch wich fixes two bugs in the gdisk package. They have been
submitted uptream but not yet been accepted.


then we don't need them if upstream did that


They didn't do anything :) Submitted only a few hours ago.




post a link to the bug report or mailing list so we can follow the progress

--
Ionuț


Re: [arch-general] [PATCH] gdisk patches

2010-07-16 Thread Mark Pustjens

On Fri, 16 Jul 2010, Ionuț Bîru wrote:


On 07/16/2010 10:11 PM, Mark Pustjens wrote:

 Hi List,

 Below a patch wich fixes two bugs in the gdisk package. They have been
 submitted uptream but not yet been accepted.


then we don't need them if upstream did that


They didn't do anything :) Submitted only a few hours ago.



--
Ionuț




Greetings/Groetjes

Mark Pustjens

--
He microwaved himself something called a Pour-On Genuine Creole Lasagne,
which said it served four portions. It did if you were dwarfs.
  (Only You Can Save Mankind)

Re: [arch-general] [PATCH] gdisk patches

2010-07-16 Thread Ionuț Bîru

On 07/16/2010 10:11 PM, Mark Pustjens wrote:

Hi List,

Below a patch wich fixes two bugs in the gdisk package. They have been
submitted uptream but not yet been accepted.


then we don't need them if upstream did that

--
Ionuț


[arch-general] [PATCH] gdisk patches

2010-07-16 Thread Mark Pustjens

Hi List,

Below a patch wich fixes two bugs in the gdisk package. They have been 
submitted uptream but not yet been accepted. The first bug fixed is that 
partition attributes are read/stored backwards. The second bug fixed is that 
when changing an undefined attribute no longer junk is displayed.


I'm not sure if this is the right place, but i sent to this list mainly because 
there is no official gpt fdisk list afai could find. Now this patch has a place 
on the net.


Cheers!

Greetings/Groetjes

Mark Pustjens

--
diff --git a/extra/gdisk/PKGBUILD b/extra/gdisk/PKGBUILD
index 0664a57..af9b44e 100644
--- a/extra/gdisk/PKGBUILD
+++ b/extra/gdisk/PKGBUILD
@@ -9,10 +9,12 @@ arch=('i686' 'x86_64')
  url="http://www.rodsbooks.com/gdisk";
  depends=('gcc-libs' 'util-linux-ng' 'popt')
  license=('GPL2')
-source=(http://www.rodsbooks.com/gdisk/gdisk-$pkgver.tgz)
+source=(http://www.rodsbooks.com/gdisk/gdisk-$pkgver.tgz
+   gdisk_attributes.patch)

  build() {
   cd $srcdir/$pkgname-$pkgver
+  patch -p1 -i "$srcdir/gdisk_attributes.patch"
make
install -D -m755 gdisk $pkgdir/sbin/gdisk
install -D -m755 sgdisk $pkgdir/sbin/sgdisk
@@ -23,4 +25,6 @@ build() {
install -D -m644 README $pkgdir/usr/share/gdisk/README
install -D -m644 NEWS pkgdir/usr/share/gdisk/NEWS
 }
-md5sums=('48740d8de518f79ae9dae7ec58068d05')
+
+md5sums=('48740d8de518f79ae9dae7ec58068d05'
+ '106a4186587ab572f6397ba1702e8d47')
diff --git a/extra/gdisk/gdisk_attributes.patch 
b/extra/gdisk/gdisk_attributes.patch

new file mode 100644
index 000..e03c957
--- /dev/null
+++ b/extra/gdisk/gdisk_attributes.patch
@@ -0,0 +1,29 @@
+diff --git a/attributes.cc b/attributes.cc
+index 527dc87..a7b2afd 100644
+--- a/attributes.cc
 b/attributes.cc
+@@ -26,6 +26,7 @@ Attributes::Attributes(void) {
+// Most bits are undefined, so start by giving them an
+// appropriate name
+for (i = 1; i < NUM_ATR; i++) {
++  temp.str(""); // empty stream
+   temp << "Undefined bit #" << i;
+ atNames[i] = temp.str();
+} // for
+@@ -75,12 +76,12 @@ void Attributes::ChangeAttributes(void) {
+do {
+   response = GetNumber(0, 64, -1, (string) "Toggle which attribute field 
(0-63, 64 to exit): ");

+   if (response != 64) {
+- bitValue = PowerOf2(uint32_t (NUM_ATR - response - 1)); // Find the 
integer value of the bit

+- if ((bitValue & attributes) == bitValue) { // bit is set
+-attributes -= bitValue; // so unset it
++ bitValue = 1 << response;
++ if (bitValue & attributes) { // bit is set
++attributes &= ~bitValue; // so unset it
+ 	cout << "Have disabled the '" << atNames[response] << "' 
attribute.\n";

+  } else { // bit is not set
+-attributes += bitValue; // so set it
++attributes |= bitValue; // so set it
+ cout << "Have enabled the '" << atNames[response] << "' 
attribute.\n";

+  } // if/else
+   } // if